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 Master Using HTML & CSS

Fadi 145

@nofear1985

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 like feedbacks, in this way i can improve my front-end abilities! feel free to give me good constructive feedback.

Community feedback

Vanza Setia 27,795

@vanzasetia

Posted

👋Hi Fadi!

👍Good job on completing this challenge! This looks good on my mobile device.

I have some feedback on this solution:

  • For the person name, I recommend to use p instead of heading tag, since it's a title of a section.
  • Also for the .text, I recommend to use blockquote instead of a div and h4.
  • Use rem unit instead of px, that way the user can control the font size based on their setting.
  • For the custom property name or variables name, I recommend to use lower case in this case. You don't have to follow this, but it's a common practice.

That's it! Hopefully this is helpful!

Marked as helpful

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