Design comparison
Solution retrospective
Conclusão do desafio "QR Code Component"
Community feedback
- @correlucasPosted about 2 years ago
👾Oi Micael, tudo bem? Parabéns pelo desafio!
Acabei de ver sua solução e tenho umas dicas pra melhorar seu código/design:
O seu componente já está com o design pronto, um bom exercício para melhorar seu código, seria criar a estrutura mais limpa o possível e reduzir seu código ao máximo. Por exemplo, você pode segurar todo conteúdo em um bloco só usando (
<main>
ou<div>
) nesse caso vamos usar<main>
por que é uma tag semântica. Se você notar, nem precisa adicionar nenhum div além da principal para segurar o conteúdo (img, h1 e p). A estrutura ideal e mais limpa possível para esse desafio é essa aqui:<body> <main> <img src="./images/image-qr-code.png" alt="Qr Code Image" > <h1>Improve your front-end skills by building projects</h1> <p>Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </main> </body>
Não é aconselhável usar IDs como seletores de CSS porque se outro elemento na página usar o mesmo estilo/semelhante, você terá que escrever o mesmo CSS novamente. Mesmo que você não tenha mais de um elemento com esse estilo agora, ele pode vir mais tarde.
👋 Espero ter ajudado e continue no foco!
Marked as helpful0 - @AdrianoEscarabotePosted about 2 years ago
Oi Micael Santiago, tudo bem?
Seja bem vindo a comunidade do front-end mentor.
Parabéns por concluir outro desafio, gostei bastante do resultado do seu desafio, mas tenho algumas dicas que julgo que você gostará:
1- Documento deve conter um marco principal, você poderia ter colocado todo o conteudo dentro da tag
main
click here2- Pagina deve conter um cabeçalho de nível 1, você poderia ter substituido o
h2
peloh1
click herePara deixar o código mais limpo tudo que você precisa é:
main
img
h1
p
O resto está ótimo!! Espero que ajude... 👍
Marked as helpful0@micaelsantiagoPosted about 2 years ago@AdrianoEscarabote Fico feliz pelo feedback e as dicas cara, valeu!
0
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