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

@sirbiel100

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


Welcome

Please, feel free to give your feedback, it's very important to help me improve!

~ Gabriel Crispim

Community feedback

P. Ricardo 2,370

@pRicard0

Posted

Dicas HTML

  • É recomendado que você deixe o atributo alt das imagens decorativas vazios, ou seja... deixa vazio o atributo alt das estrelinhas.
  • Recomendo utilizar a tag <em> ao invés da tag <span> para o texto "Verified Buyer". Essa tag é a tag semântica para textos que estão sendo enfatizados, que se enquadra nesse caso. O texto dentro dessa tag serão lidos com ênfase verbal, para deixar bem claro.
  • Achei esquisito que a tag header engloba um comentario, a main engloba outro comentário e tem a tag bottom para englobar outro. Acho meio esquisito. Ao invés de utilizar section para englobar tudo isso, por que não troca por main e cada tag para englobar esses comentários por section? É só uma ideia, fica ao seu critério.

Acho que só isso mesmo, sua estilização ficou perfeita.

Marked as helpful

1

@sirbiel100

Posted

@pRicard0 Obrigado pelo feedback Ricardo! E sim, concordo com sua ideia, o código fica bem mais fácil entender do jeito que você disse. 👊

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