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 flexbox model and media query tool

@shivam5750

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Should I use flexbox or CSS grid to design the layout ...and kindly give your suggestion how can i improve my designs

Community feedback

T
Mike Hayden 1,005

@mickyginger

Posted

Hi Shivam, this looks great!

I think you could try different HTML elements for your structure. I would recommend using main where you've used section, and section for the .container element. There is also a header element that might be good for the .head element.

The layout is a little wide on my large screen, so it's probably worth setting a max-width to the main element so that it never stretches more than say 1200px.

As far as layout is concerned, I'm not sure there's a hard and fast rule about using flexbox or CSS grid. For a simple layout like this I would probably use flexbox, but it might be fun to try it both ways and see which you prefer. As with any coding, simpler is always better!

Hope that helps!

1

@shivam5750

Posted

@mickyginger Hey! thanks for your feedback... next time i will try to improve and will folow your instructuion...

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