Design comparison
Community feedback
- @R3ygoskiPosted 6 months ago
Olá David, parabéns por mais um projeto concluído, esse ficou bem próximo do Design, meus parabéns!
Gostaria de dar aqui uma dica, quanto a parte semântica do seu HTML, ela como sempre está perfeita, é só apenas um pequeno detalhe que notei. O mais correto para seus
<button type="button">
, é eles serem parte de uma<ul>
e estarem em uma<li>
, isso porque são uma lista de ícones de redes sociais, além disso, ao invés da tag<button>
o que mais se encaixa aqui seria a tag<a>
, porque esse botões tendem a fazer o papel de um link. O<button>
serve para quando vamos realizar uma ação dentro da nossa página.E uma outra sugestão ainda sobre acessibilidade, é que ao utilizar o
<a>
, e não ter texto dentro dele, utilize umaria-label="nomeDoLink"
, que dessa forma você compensa na parte de acessibilidade.[Outra coisa que notei, mas que é detalhe pequeno, é que o sombreamento do design é um pouco mais forte no card, e um pouco mais pra baixo, digo isso caso você pretenda modificar a solução.
E mais uma vez, parabéns, seus projetos são muito bons semanticamente (em outras partes também é), continue assim com esse bom trabalho. Se ficou alguma duvida quanto ao que falei, já sabe, só comentar abaixo.
Marked as helpful1@dvdgdPosted 6 months ago@R3ygoski Muito obrigado pelo feedback, eu pretendo modificar a solução por conta da semântica do HTML, realmente não faz sentido usar um
<button>
se estou levando o usuário a outra página.1
Please log in to post a comment
Log in with GitHubJoin 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