Design comparison
Solution retrospective
To fit the qr code photo, use the CSS property 'object fit' and to help with the responsive layout use the properties max-width and max-height.
Community feedback
- @danielmrz-devPosted 9 months ago
Fala @Iarley-01!
Seu projeto está excelente!
Tenho duas sugestões:
📌 Primeira: Use
<main>
pro conteúdo principal ao invés de uma<div>
.Tags como
<div>
e<span>
são exemplos típicos de elementos HTML não-semânticos. Elas servem apenas como containers para o conteúdo, mas não indicam qual o tipo desse conteúdo e nem o papel que ele desempenha na página.📌 Segunda: Também por questões semânticas, use
<h1>
para o título principal ao invés de<h2>
.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.
Essas mudanças de tag geram pouco ou 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 helpful0 - As tags
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