Skip to content
Submitted over 5 years ago

Reponsive landing page using CSS Grid

Jon Colville•80
@jonatronc
A solution to the Four card feature section challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hello,

I would love any feedback on the code structure and advice on better techniques.

Thanks!

Code

Please log in to post a comment

Log in

Community feedback

  • Matt Studdert•13,611
    @mattstuddert

    Hey Jon, awesome work on this challenge. Your solution and code both look great! 👍

    As far as better techniques go, I'd recommend avoiding IDs as CSS selectors. They're overly specific and also can't be reused on the page. Instead, stick to class, attribute, pseudo and type selectors. This will help massively as you build larger projects.

    Also, on a future project, have a go at using min-width media queries instead of max-width. Working mobile-first and using min-width queries often leads to less CSS code and it also has the benefit of loading in fewer styles for mobile users.

    I hope these pointers help. Keep up the great work! 👍

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