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

Bilel 70

@Bilel-mwihbi

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 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 34,090

@AdrianoEscarabote

Posted

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
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