@alberto-rj
Posted
👋 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 helpful