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 section

Renzo 175

@renzo4web

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


I started with the mobile layout first like always, my purpuse with this project was to conoslidate the knowledge of grid and flexbox.

Community feedback

P
tediko 6,680

@tediko

Posted

Hello, Renzo! 👋

Congrats on finishing another challenge! Your solution responds well. Here's my small suggestions:

  • Use <h1> for .proof__title instead of <h4> since it is your primary heading.
  • Change the alt attributes for the testimonial avatars, as they don't have any extra context for screen reader users. Changing to alt="" will make screen readers skip those avatar images.
  • You repeat your HTML code with stars images which is unnecessary. Find the way to use only one star image for a .stars. You can do it easily by using background-repeat approach in your css.

Good luck with that, have fun coding! 💪

1
Renzo 175

@renzo4web

Posted

Thanks for the review

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