HTML and CSS and Grid

Community feedback

• 

  Zineb Boutaa•750

  @zineb-BouPosted about 3 years ago

  Hi Mohammed, I noticed couple of things in your solution, the desktop layout doesn't show up to me, I think you chose the mobile first approach, then using media queries to set up the desktop, so check again the min-width. keep coding...

  Marked as helpful

  1

  ---

  

  Mohammed Hany•140

  @MohammedHany1Posted about 3 years ago

  @zineb-Bou Thank you for your helpful reply Zineb! I have checked the min-width again and changed a couple of things. could you please check the site again to see if the problem is gone?

  0

  

  Zineb Boutaa•750

  @zineb-BouPosted about 3 years ago

  @MohammedHany1 Hey Mohammed, I just checked again your solution, the problem was fixed, Good job,

  0
• 

  Shubhankar Sengupta•285

  @Shubhankar-SenguptaPosted about 3 years ago

  It looks great

  Marked as helpful

  1
• 

  0rGaan1c•150

  @0rGaan1cPosted about 3 years ago

  It looks good on mine as well, probably an issue with frontendmentor, maybe try submitting it again.

  Marked as helpful

  1

  ---

  

  Mohammed Hany•140

  @MohammedHany1Posted about 3 years ago

  @0rGaan1c I have just submitted it again but the problem is still there. I just want to check if it happens on a specific screen size or something like that so I can fix the bug If it is there. but thanks for your help :)

  0
• 

  Bian Goole•125

  @BianGodePosted about 3 years ago

  It looks good. You can improve the width and height of the grid items. Keep on going!

  0