Design comparison
Community feedback
- @correlucasPosted about 2 years ago
👾Fala Matheus, tudo bem? Parabéns pelo seu primeiro desafio e seja bem vinda a comunidade do Frontend Mentor!
1.Substitua o
div
por<main>
(toda página deve ter um landmark principal e também um título principal h1).2.Seu código tá um pouco longo, da pra vc reduzir ele:
<body cz-shortcut-listen="true"> <div class="container"> <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> </body>
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).<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>
👾Minha avaliação para sua solução: ⭐⭐⭐⭐⭐
👋 Espero ter ajudado e continue no foco!
Marked as helpful1
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