Design comparison
Community feedback
- @R3ygoskiPosted 7 months ago
Olá @laladrack, parabéns pelo seu projeto, ele ficou semelhante ao design proposto.
Gostaria de dar algumas dicas e recomendações, no seu HTML, notei que você utilizou a tag
<body>
como o card, isso não está errado, mas geralmente utilizamos a<body>
como o fundo da nossa página e não um componente individual como nesse caso, o card. Para isso você pode usar alguma outra tag, como<div>
ou as tags semânticas que pessoalmente eu recomendo muito, que nesse caso poderia ser<section>
ou<article>
.E também notei duas coisas relacionadas a acessibilidade, que a primeira é na sua tag
<img/>
que ela não possui um atributoalt
, nesse caso, como o qr-code é uma imagem importante, é interessante que nela seja atribuído umalt
com um texto descrevendo o que é a imagem. Agora em casos de ícones, como botão de + por exemplo, não é necessário passar um textoalt
, mas é recomendável que se tenha pelo menos o atributoalt=""
vazio.Outra coisa foi sobre a hierarquia dos headers, no seu projeto notei que você utilizou a
<h2>
, acredito que você fez isso pois a<h2>
se assemelha muito ao design proposto, mas isso pode causar problemas de acessibilidade e semântica, o mais correto seria utilizar uma<h1>
.Alguns Links sobre o que falei:
E novamente, parabéns pelo seu projeto, ele ficou realmente muito semelhante ao design proposto, caso algo que eu tenha dito não tenha ficado claro, por favor, pergunte aqui embaixo que tentarei ajudar da melhor forma possível.
Marked as helpful0
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