Design comparison
Community feedback
- @AdrianoEscarabotePosted about 2 years ago
Oi Jamily, tudo bem?
Seja bem-vinda a comunidade do front-end mentor. Você fez um ótimo trabalho neste desafio, mas tenho algumas dicas que julgo que você gostará:
1- Documento deve ter 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 trocado o
h2
peloh1
click herePara ter um código mais limpo, você poderia ter feito:
main
img
h1
p
O resto está ótimo, a responsividade do projeto ficou ótima! Espero que ajude... 👍
1 - @correlucasPosted about 2 years ago
👾Fala Jamily Aleixo, tudo bem? Parabéns pelo seu primeiro desafio e seja bem vinda a comunidade do Frontend Mentor!
O seu componente já está com o design pronto, um bom exercício pra 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 semantica. Se você notar, nem precisa adicionar nenhum div além da principal pra segurar o conteúdo (img, h1 e p). A estrutura ideal e mais limpa possível pra esse desafio é essa aqui:SUA ESTRUTURA ANTES DA REDUÇÃO
<div class="conteiner"> <div class="card"> <img src="images/image-qr-code.png" alt="image-qr-code"> <div class=" text"> <h2>Improve your front-end skills by building projects</h2> <p>Scan the QR code to visit Frontend Mentor and take your coding skills to the next level</p> </div> </div> </div>
CÓDIGO DEPOIS DA REDUÇÃO:
<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>
Para reduzir o CSS, você pode excluir todas as classes e selecionar todos elementos no CSS com o seletor direto para cada um deles, nesse caso (img, div, main, h1 e p). Faltou também colocar o
alt
text pra imagem ser reconhecida por leitores de tela.👋 Espero ter ajudado e continue no foco!
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