QR Code Component

Hi Diego,

Congratulation on completing your first challenge here in frontend mentor. Your solution looks great. I have some suggestions regarding your solution:

• The alternate text should indicate where the Qr code navigate the user : like QR code to frontend mentor not describes the image.

• Consider using min-height: 100vh instead of height: 100vh to the body , that let the body grows taller if the content of the page outgrows the visible page.

• It's recommended to include a git ignore. This came with your starter files. It's less important in this challenge but will become extremely important as you move onto larger projects with build steps.

• There are a lot the arguments against the 62.5% font size trick ,it state that you should never change the root font size because it harms accessibility.

Overall, Excellent work! Hopefully this feedback helps.

Fala Diego, beleza? Parabéns pelo primeiro desafio!

Eu vi que você foi bem minimalista na estrutura do HTML, de fato sua solução foi uma com o código mais limpo de todas que vi. Eu usaria <main> no lugar do article pra evitar o erro de acessibilidade que vai aparecer pra você porque toda página precisa de um main mesmo nesse caso sendo um componente.

É se você quiser limpar mais ainda o código, dá pra tirar todas as classes e manipular o CSS todo com o seletor direto de cada elemento já que são únicos, main, article, img, h1 e p.

Parabéns, mandou muito!