Article Preview Component using position and flex

Community feedback

• 

  Bernardo Poggioni•5,360

  @R3ygoskiPosted 7 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 um aria-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 helpful

  1

  ---

  

  David Dutra•230

  @dvdgdPosted 7 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