Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Social proof section challenge

P
Gab 170

@gab-holik

Desktop design screenshot for the Social proof section coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What specific areas of your project would you like help with?

Any feedback is appreciated. Thank you 🙂

Community feedback

Bryan Li 3,550

@Zy8712

Posted

Hi there! Your site looks great and follows the original design closely. Nice work!

Some things I'd suggest you to change with your code:

  • for decorative images you should leave the alt description empty like alt="" as screen readers will read anything that is in the alt descriptions. Meaning in your case it'll read things like "one star", "two star", "Customer image of Irene Roberts", etc
  • alt descriptions for <img> tags should mainly be used for images that convey some important information

Aside from that your code structure looks great. Hope you find this feedback useful 👍

Marked as helpful

1

P
Gab 170

@gab-holik

Posted

Hi Bryan,

Thank you so much for your feedback! I thought all images need an alt text but that actually makes perfect sense that decorative ones shouldn't have any as they can be super distracting. Thank you!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord