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

Responsive Social proof section

TheWarley 120

@TheWarley1

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


Sigh! This was tough!. Glad I could figure it out. I'm open to any suggestions/ corrections.

ty❤️

Community feedback

Arne 1,110

@Dudeldups

Posted

Hello! I'm a beginner myself but I see some things to improve: You are wrapping many elements in divs, e.g. the <p> below the <h1>. You can just give the paragraph a class name and remove the div, since you don't use that kind of "wrapper" elsewhere on the site. You are using id="vb" for the verified buyer paragraph. Use classes there, ids are supposed to be unique. And try to be more descriptive: vb is hard to understand for other ppl reading your code (especially in the CSS file), as well as your future self :D

Marked as helpful

2

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