Flex, Gird & Media Queries

Community feedback

• 

  arey•350

  @arey-devPosted over 2 years ago

  Great work! I like your layout between 768px - 992px!

  Here's my few comments.

  • You can write @media (max-width:992px) just once, and put all the CSS rules that you want to fall under that viewport size.
  • And maybe, try to maintain the same spacing that you have from 768px - 992px on the top and bottom of the page under 768px viewport width.

  Marked as helpful

  0

  ---

  

  P

  Islam Soliman•440

  @simokitkatPosted over 2 years ago

  @arey-dev

  Thank you for your feedback!

  I checked your solution of the same challenge and I liked it very much too.

  I thought of putting all the CSS rules of @media (max-width:992px) at once, but I thought that in case that was a real project , it would be easier to edit the media queries if they are written right under the original ones. Not sure if that's the best practice though.

  1

  

  arey•350

  @arey-devPosted over 2 years ago

  @simokitkat It's nice that you are writing your code under the presumption of it being a real and bigger project. You can read this guide. This will help you organize your css when doing more complex projects.

  1