using flexbox, media query

Community feedback

• 

  Lazar Nikolov•260

  @nikolovlazarPosted almost 4 years ago

  Great job Esraa! 🙌

  I could suggest you wrap your content in a "container" div to keep everything tidy. Also, pay attention to the small details, your background is different, you're missing the shadows, and the spacing between the cards is not consistent. Noticing these details will become easier and easier as you code, so don't worry, you're on the right path! For reference, you can research the "Masonry Grid" layout and its responsiveness. It'll give you an idea on how to refactor your solution and make it "top shelf".

  This is awesome Esraa! Keep up the great work!

  0

  ---

  

  esraagamal•600

  @EsraaGamal-22Posted almost 4 years ago

  @lazarnikolov94 Thanks a lot for your opinion 😃 , can you review my code after the update?

  0