Design comparison
SolutionDesign
Community feedback
- @HassiaiPosted almost 2 years ago
Replace <h2 class="title"> with <h1> to fix the accessibility issue. There is no need to give html and body a height of 100%. add min-height:100vh; to the body
Give .product a padding value of 1rem for all the sides and reduce the gap value to 1rem/
You forgot to add a media query for the mobile design, in the media query give .product a width of 80%
Hope am helpful Happy coding
Marked as helpful0
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord