Skip to content
Submitted over 3 years ago

Social proof section

Renzo•175
@renzo4web
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Social proof section
  • HTML
  • CSS
1newbie

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

  • tediko•6,680
    @tedikoPosted over 3 years ago

    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
    @renzo4webPosted over 3 years ago

    Thanks for the review

    0

Please log in to post a comment

Log in with GitHub

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