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 landing page using CSS Grid with mobile popup Navbar

#sass/scss
Rashi 165

@shangum

Desktop design screenshot for the Sunnyside agency landing page coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
2junior
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi, For this challenge I was practicing with CSS Grid but am still new. Any thoughts on the way I decided to structure the grids would be helpful. I want to make sure that my CSS design patterns are heading in the right direction (as opposed to leaning towards bad habits). I used grids to layout certain sections instead of creating an encapsulating mega grid. The three major grids were the 6 celled one directly beneath the header section, the client testimonials section, and the pseudo image gallery near the bottom.

Also, I used “@media screen” media queries for the different screen sizes. They apply themselves correctly when I use Chrome Dev Tools to change to different screen sizes. I would also like to be able to apply these rules when a desktop user manually resizes their browser window. Is this possible using only Sass/CSS or would I need to handle this with JavaScript?

If any other things stand out that I could improve on, don’t hesitate to let me know.

I know it can be a bit time consuming to review someone else’s code and work so thanks to anyone who gave it a look (both now and in the past).

Thanks

Community feedback

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