Submitted over 1 year ago

PAgina com QR code responsiva

cube-css, jss, lighthouse, sass/scss, rxjs

Raphael Carvalho•60

@RaphaelCarvalh

A solution to the QR code component challenge

Preview Code

Solution retrospective

What are you most proud of, and what would you do differently next time?

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

Code

Please log in to post a comment

Community feedback

Alberto José•630
@alberto-rj
Posted over 1 year 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 helpful

Join 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