Página de QR code que leva ao frontend mentor
Design comparison
Community feedback
- @StroudyPosted about 2 months ago
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! 🎯
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