@correlucas
Posted
👾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 helpful