Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Responsive layout with grid

@abdallaelnjoomi

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Each one is commenting, has place in my heart. please give me your feedback.

Community feedback

hitmorecode 6,230

@hitmorecode

Posted

Nice well done. Just a few tips

  • Remove height: 100vh; on the wrapper and place it on the body and make it min-height: 100vh;.
  • In the "Why us" section you used a <p> and <br> tag to separate the text. You could have done this with
<ul>
    <li></li>
    <li></li>
    <li></li>
</ul>
  • The button should be interactive
0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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