CSS Grid Layout

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hello there, Yeimy! 👋

  Good job on this challenge! 👏 Your solution looks great and responds very nicely (I like the intermediary layout that you added)! 👍

  The only tip I have is that you don't need to add the word "image" to the alt text for the images on the page (since the fact that the element is an image will already be announced by screen readers and so including "image" in the alt text will be a bit redundant). Very small thing, though! 😉

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  1

  ---

  

  Yeimy•365

  @Yeimy7Posted over 3 years ago

  Hello @ApplePieGiraffe!

  Thank you very much for your comments, I will consider your suggestion =D

  1
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  ⭐⭐⭐⭐⭐ Great Job and very nice transition from desktop to mobile

  Marked as helpful

  1

  ---

  

  Yeimy•365

  @Yeimy7Posted over 3 years ago

  Hello Patrick!

  Thank you for your comment!

  1
• 

  Anurag Singh•1,265

  @exist08Posted over 3 years ago

  Excellent , Everything looks good. Also Tablet View is good. Keep Learning !!!

  0