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 component (grid, flexbox)

@Ritesh-Virulkar

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


please provide some suggestions to improve.

Community feedback

@Mar1362

Posted

Hi, here is some suggestions:

  • i do not think that you used a right semantic in your html code, i am talking about the use you do with the article and the use you do with your unordered list so check out this

  • your icon-stars are just decorative, it is just the text that go with them which is important so you have to add aria-hidden='true' attribute or role='presentation'. personnally i added them as background-image you can go check how i did ( i added some animation too in mine :) ) and apart those images all the rest are important and there alt attribute should be filled correctly

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