Design comparison
Solution retrospective
nenhum
What challenges did you encounter, and how did you overcome them?até o momento nada
What specific areas of your project would you like help with?nenhuma
Community feedback
- @alberto-rjPosted 6 months ago
👋 Hi, Raphael!
👏 Congratulations on investing your time and taking your frontend skills to the next level with the QR code component challenge.
I have a few suggestions that could help you improve your solution to this challenge:
On line 14 of your
index.html
file: Instead of using the<p>
tag to define the most important title of the page, you could use the<h1>
tag, as it already has this semantic meaning. Learn more👎 Don't
<p class="texto">...</p>
👍 Do
<h1 class="texto">...</h1>
On line 15 of your
index.html
file: For the body of the text, instead of using the<texto>
tag, you could use the<p>
tag. This is because the<texto>
tag is not a valid HTML tag. You can use the W3C Markup Validation Service to verify this by clicking here.👎 Don't
<texto class="caixa-menor">...</texto>
👍 Do
<p class="caixa-menor">...</p>
Your solution has contrast errors: Low-contrast text is difficult or impossible for many users to read. You could avoid these issues by using the Accessibility Insights for Web extension.
Keep learning, coding and sharing! 🌟
Happy coding! 🚀
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