Skip to content
Submitted almost 2 years ago

Social proof section

Bilel•70
@Bilel-mwihbi
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

Hello everyone 👋, hope you all are doing well !

I'd like to see how you solved the rating stars part. For typography i used clamp() but i dont think it worked well also I had a difficulty in positioning the background because it looked unclear.

Feel free to leave any suggestions/advice thanks ! 👍

Community feedback

  • Adriano•38,120
    @AdrianoEscarabotePosted almost 2 years ago

    Hi Billel, how are you? I really liked the result of your project, but I have some tips that I think you will enjoy:

    You have used <br> , using <br> is not only bad practice, it is problematic for people who navigate with the aid of screen reading technology. Screen readers can announce the presence of the element. This can be a confusing and frustrating experience for the person using the screen reader. You can read more in MDN.

    Prefer to use max-width on the element when you want to do a line break.

    The rest is great!

    I hope it helps... 👍

    1

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