Design comparison
Solution retrospective
Opa, comecei a estudar Flexbox a algumas semanas e decidi fazer esse desafio do QR code, qualquer feedback eu ficarei grato :))
Community feedback
- @danielmrz-devPosted 10 months ago
Fala @msumii!
Seu projeto está excelente!
Tenho duas sugestões:
- Por questões semânticas, e por ser o título principal da tela, você pode substituir o
<h2>
por<h1>
. Ao contrário do que a maioria das pessoas pensa, a diferença entre os HTML headings não é só sobre o tamanho e peso do texto.
📌 As tags
<h1>
a<h6>
são usadas para definir títulos em HTML.📌
<h1>
define o título mais importante.📌
<h6>
define o título menos importante.📌 Use apenas um
<h1>
por página - isso deve representar o título principal de toda a página. Além disso, não pule os níveis de título - comece com<h1>
, depois use<h2>
e assim por diante.- Ainda sobre HTML semântico, use
<main>
pro conteúdo principal ao invés de uma<div>
.
Essas mudanças de tag não geram nenhum impacto visual mas tornam o seu código HTML mais semântico e melhoram a otimização SEO e acessibilidade do projeto.
Espero que ajude!
Fora isso, ótimo trabalho!
Marked as helpful1 - Por questões semânticas, e por ser o título principal da tela, você pode substituir o
Please log in to post a comment
Log in with GitHubJoin 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