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 / HTML CSS

yudizβ€’ 180

@yud11z

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


All feedbacks are welcome, Regards

Community feedback

Lucas πŸ‘Ύβ€’ 104,540

@correlucas

Posted

πŸ‘ΎHello @yud11z, congratulations for your new solution!

You've done a really good work building this solution, indeed I don't have nothing to say about the design and the html structure, everything is working. The only thing you should consider is to use semantic tags to wrap big blocks of content, since divs doesn't have a meaning, its not the best pick for blocks like the cards, you can replace the card div with <article> and the text content that is a quote with the tag <blockquote> thats the exact tag that describes its content, in this case the quote.

Here's a complete guide for HTML semantic TAGS: https://www.w3schools.com/TAgs/default.asp

πŸ‘‹ I hope this helps you and happy coding!

0

yudizβ€’ 180

@yud11z

Posted

@correlucas Thanks for the feedback, I really appreciate that. Regards

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