Social Proof Section

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted almost 2 years ago

  Hey, Mirza Monirul Alam! 👋

  Nice work on this challenge! 👍 Your solution looks good and is responsive! 👏

  You don't need to add role="presentation" to the star icon images since the img tag is already semantic and you've left the alt attribute empty for those images (good choice, btw) meaning screen readers will ignore those elements.

  Another small tip I have for you is to add min-height: 100vh to the body element so that it will always be at least as high as the user's screen and there won't be any empty space to the bottom of the design.

  Besides that, you should add a favicon to the site. There should be one in the starter files for this challenge that you can use. 😉

  Hope you find this helpful. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  0