Skip to content
Submitted about 4 years ago

Single price grid component - HTML, CSS Grid, Media query

Nene16•95
@Nene16
A solution to the Single price grid component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

My second try with css grid.

As always, If someone could give me some feedback to improve my skills, I would really appreciate it.

Code

Please log in to post a comment

Log in

Community feedback

  • Vanza Setia•27,715
    @vanzasetia

    👋Hi Nene! My name is Vanza!

    I have some feedback on this solution, that may improve this solution:

    • For the container, I recommend to use max-width as a default style, not inside @media all and (min-width:1024px). The problem with putting the width inside this media query is that, it will only applied when the screen is 1024px and above. So, when the screen size is 960px the container will become too wide.
    • For the zero number, the best practice is that never put a unit on zero number, because 0 is equal to 0px, 0%, 0rem and etc.

    That's it! Hopefully those feedback will improve your skill!

    Happy Coding and Keep Learning!

    Marked as helpful

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