Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Card QR code

@Gabriel-Vlas

Desktop design screenshot for the QR code component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Qualquer Feedback é bem-vindo

Community feedback

Lucas 👾 104,420

@correlucas

Posted

👾Oi @Gabriel-Vlas, tudo bem? Parabéns pelo desafio! Seja bem vindo a comunidade do Frontend Mentor

Acabei de ver sua solução e tenho umas dicas pra melhorar seu código/design:

Ao invés de usar <h2> pro bloco contendo o título principal, ao invés use <h1>, porque cada página precisa de um h1 pro título principal e siga a sequência se você adicionar outro título como h1, h2 , h3…

Use <main> ao invés de usar uma <div> para conter todo conteúdo do componente. Desse jeito você melhora a semântica e indica qual é o bloco mais importante de conteúdo da página. Lembre-se que div é só um bloco de conteúdo sem significado e que cada página precisa do <main> para mostrar qual o bloco mais importante.

Use unidades relativas como rem ou em em vez de px para melhorar seu desempenho redimensionando fontes entre diferentes telas e dispositivos. Para fazer um site mais acessível, o ideal seria usar rem em vez de px. O REM não se aplica apenas ao tamanho da fonte, mas também a todos os tamanhos.

👋 Espero que essas dicas te ajudem e que você continue no foco!

Marked as helpful

1

@Gabriel-Vlas

Posted

@correlucas Muito Obrigado por esse feedback, ficarei mais atento a essas dicas!!

0

Please log in to post a comment

Log in with GitHub
Discord logo

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