Responsive intro section with dropdown navigation using flexbox

Community feedback

• 

  Prabhash Ranjan•2,540

  @besttlookkPosted over 2 years ago

  Hi, Nice work! Everything looks great. Here are a few points I like to add:

  • On clicking overlay, sidemenu should close(for small screen)
  • Dropdown should also also while clicking outside(for large screen)
  • For large screen it would have looked good when the content were covering the whole view-port height.

  Good luck,

  Happy coding

  Marked as helpful

  1

  ---

  

  Ricky•470

  @pyaetheiNPosted over 2 years ago

  @besttlookk Thank you for your feedback! I'll learn more about JavaScript events that you mentioned and I've fixed the content covering issue for larger screens.

  0
• 

  Nate•320

  @Ging3rmintPosted over 2 years ago

  Nice work. rather than putting media query for each and every bigger screen size, what we usually do is to set a max-width to your container then use margin: 0 auto to centralise it.

  1

  ---

  

  Ricky•470

  @pyaetheiNPosted over 2 years ago

  @Ging3rmint yeah I added margin: 0 auto but that's not the case right now, I’m coding on laptop so to match the desktop screenshot, the image would be too big on screens between 968px and 1200px so I had to make another media query...

  0