Skip to content
Submitted about 1 year ago

Responsive layout with grid

abdallaelnjoomi•130
@abdallaelnjoomi
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Single price grid component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

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

Community feedback

  • hitmorecode•6,250
    @hitmorecodePosted about 1 year ago

    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

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