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

Mobile first approach using Sass

@dralm3ida

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


Hello!

Any feedback is appreciated.

Thank you.

Community feedback

T
Grace 29,910

@grace-snow

Posted

Hi

This looks really good.

Only important change to make is to alt text on your images

  • alt="icon-star"> should be alt="" aria-hidden="true". This is because these are not meaningful images and the writing says 5 stars already.
  • and on the testimonial images the alt attribute value should be eahh persons name. It has to be a readable phrase or sentence, not a hyphenated word like that. And it should never include words like "image" or "photo" as the image element is already announced as an image anyway.

I hope these tips are useful ☺

1

@dralm3ida

Posted

@grace-snow, thank you for your feedback. It will be corrected.

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