Design comparison
Solution retrospective
Hello everyone! If you have any general advice for the whole project, I'm trying to learn more about frontend development, more specifically, HTML, CSS and JS to build some static sites.
Any review is a good review!
Community feedback
- @danielmrz-devPosted 11 months ago
Fala, @abxsantos!
Seu projeto ficou excelente!
Tenho apenas 2 sugestões:
- Primeira: Substitui aquele
h2
por umh1
. Muita gente acha que a única diferença entre os headings (h1 até h6) é apenas o tamanho, mas não é. Eles tem valor semântico e podem ajudar a melhorar a otimização SEO da página e também a acessibilidade.
Se liga nessa descrição da W3Schools sobre os headings:
As tags
<h1>
a<h6>
são usadas para definir cabeçalhos 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/assunto 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.- Segunda: Dá uma olhadinha no arquivo
style-guide
que vem junto com o projeto pra verificar afont-family
recomendada. Se minha memória não está errada, acho que a fonte pra esse projeto é a Figtree.
Fora esses detalhes, meu amigo, você fez um ótimo trabalho!
Espero que ajude! E Feliz Ano Novo 🎉😁
Marked as helpful0@abxsantosPosted 11 months ago@danielmrz-dev muito obrigado!!
Uma dúvida que fiquei em relação aos headings. Entendo que para o caso, como tenho apenas esse componente na página, usar o
h1
. Mas pensando que esse componente vá ser utilizado em uma página mais complexa, como seria o modo ideal de definir a tag a ser utilizada para o "título" do card? Faria sentido não usar um heading, mas sim um span nesse caso?0 - Primeira: Substitui aquele
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