Design comparison
SolutionDesign
Solution retrospective
What specific areas of your project would you like help with?
Any feedback is appreciated. Thank you 🙂
Community feedback
- @Zy8712Posted 7 months ago
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 helpful1@gab-holikPosted 7 months agoHi 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 - for decorative images you should leave the alt description empty like
Please log in to post a comment
Log in with GitHubJoin 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