QR code component

Hello there! Good job in this challenge. Viewing the solution, I think you should consider the following as well…

• Instead of a div, alternatively wrap the content with a main tag . Use semantic tag, for HTML structures, refer it with this one Semantics
• Remove height: 68vh; in .qrcode-conteudo rule set. Add padding: 1rem;
• Remove height: 250px; and adjust width into width: 100%; in .img-principal rule set
• Add margin-top: 0; in the .text rule set
• Add padding: 1rem; in the attribution rule set

Above all, Well done! Keep it up and Hope this is helpful!

Hello there! 👋

Congratulations on finishing your challenge! 🎉

I have some feedback on this solution:

• Always Use Semantic HTML instead of div like <main> <header> <footer> , etc for more info

• reduce the height of the qr-code to 50vh

i hope this is helpful and goodluck!

Hey @Gabriel-Leao, I like this.

Hay ! Gabriel Good Job on challenge

These below mentioned tricks will help you remove any Accessibility Issues

-> Add Main tag after body <main class="container"></main>

-> Learn more on accessibility issues

If this comment helps you then pls mark it as helpful!

Have a good day and keep coding 👍!