Design comparison
SolutionDesign
Solution retrospective
Would love to hear your feedbacks ❤️..
Community feedback
- @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 - @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@saakarxPosted over 3 years ago@tediko Thanks for the feedback, will keep that in mind not doing that again. 😊
0
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