@Stroudy
Posted
Exceptional work! You’re showing great skill here. I’ve got a couple of minor suggestions that could make this stand out even more…
-
Avoid using
id
selectors for styling in CSS because they are too specific and hard to override, making your styles less flexible and maintainable. Instead, use class selectors (.
), which are reusable and more manageable, allowing for better control over your styles and easier updates. -
Overusing
<div>
tags, known as "divitis," leads to cluttered code, poor semantics, and reduced performance. Instead, use appropriate semantic elements (like<header>
,<section>
, etc.) to improve readability, accessibility, and SEO. Keep HTML clean and minimal to ensure maintainability, scalability, and better CSS structure.
<div id="imagem">
<img src="images/qr-code-p.png" alt="QR Code">
</div>
- You have a
section
and amain
doing the same job, You can remove section to save any confusion.
<main>
<section>
<div id="imagem">
<img src="images/qr-code-p.png" alt="QR Code">
</div>
<div id="info">
<h1>Melhore suas habilidades de front-end construindo projetos</h1>
<p>Leia o QR code para visitar o Frontend Mentor e leve suas habilidades de programação para o próximo nível</p>
</div>
</section>
</main>
You’re doing so well, and I hope this feedback is helpful! Keep honing your skills and remember to enjoy the process—you’re on a great path. Stay motivated, and happy coding! 🎯