Social Proof Section

Community feedback

• 

  T

  Grace•31,270

  @grace-snowPosted over 3 years ago

  All great feedback above. You should definitely have empty alt attribute values on all those star images too. At the moment it would be very tedious for a screenreader to read out "image, star, image, star, image..." etc.

  0

  ---

  

  SaakarX•320

  @saakarxPosted over 3 years ago

  @grace-snow Thanks for checking it out, will fix that 😊.

  0
• 

  P

  tediko•6,680

  @tedikoPosted over 3 years ago

  Hello, Saakar Gogia! 👋

  Good effort on this challenge! In addition to palgramming feedback I would suggest you to:

  • Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
  • Change the alt attributes for the .user__profile-image avatars, as they don't any extra context for screen reader users. Changing to alt="" will make screen readers skip those avatar images.

  Good luck with that, have fun coding! 💪

  0

  ---

  

  SaakarX•320

  @saakarxPosted over 3 years ago

  @tediko Thanks for the feedback, will keep that in mind not doing that again. 😊

  0
• 

  P

  Patrick•14,265

  @palgrammingPosted over 3 years ago

  Looks good but around 600px you have elements going off the screen you need to fine tune your mobile to desktop transition

  0

  ---

  

  SaakarX•320

  @saakarxPosted over 3 years ago

  @palgramming Thanks for the feedback, will look forward to fix that.

  0