Responsive landing page using css flexbox model and media query tool

Community feedback

• 

  T

  Mike Hayden•1,005

  @mickygingerPosted over 3 years ago

  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

  ---

  

  Shivam Kumar Gupta•50

  @shivam5750Posted over 3 years ago

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

  0