QR Code Component

Community feedback

• 

  PhoenixDev22•16,950

  @PhoenixDev22Posted over 2 years ago

  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.

  Marked as helpful

  1
• 

  Lucas 👾•104,340

  @correlucasPosted over 2 years ago

  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!

  Marked as helpful

  1

  ---

  

  Diego•80

  @diegopereira0Posted over 2 years ago

  @correlucas

  Fala, Lucas! Tudo certo, e você?

  Sim, eu percebi assim que subi o código e apareceu o erro aqui 😂 Não sabia! Mas valeu pela força, eu já corrigi e dei um push no git.

  Muito obrigado, meu caro!

  1