@huyphan2210
Posted
Hi @DerRight,
I checked out your solution, and here are a few thoughts:
- Layout on Mobile Devices: The page layout doesn’t look as expected on mobile devices. I'd recommend using media queries to improve responsiveness. Additionally, exploring a mobile-first approach might help—designing for mobile devices first and scaling up for desktops.
- HTML Semantics: Your HTML structure is quite semantic—nice work! However, there are some areas for improvement. For example, since
.card
is the only child of themain
tag, usingmain
directly for.card
instead of anarticle
could simplify the structure. - Heading Tags: The
.title
class seems to represent a heading, so using anh1
tag instead of ap
might be more appropriate. - Form Structure: For
.input
, using aform
element with a submitbutton
could be beneficial. This allows users to press Enter to submit rather than having to click thebutton
. You might want to look into handling form submissions for more usability.
Hope this feedback helps!