Submitted over 3 years ago

Social Proof Section

SaakarX• 320

@saakarx

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Social proof section

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

Would love to hear your feedbacks ❤️..

Community feedback

Grace• 29,310

@grace-snow

Posted 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.

SaakarX• 320

@saakarx

Posted over 3 years ago

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

tediko• 6,680

@tediko

Posted 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! 💪

SaakarX• 320

@saakarx

Posted over 3 years ago

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

Patrick• 14,285

@palgramming

Posted 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

SaakarX• 320

@saakarx

Posted over 3 years ago

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

Please log in to post a comment

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