@danielmrz-dev
Posted
Fala Gustavo!
Seu projeto ficou ótimo!
A parte responsiva ficou certinha. Tenho duas sugestões:
-
Adiciona um
hover effecte umcursor: pointerno título principal do card pra mudar a cor pra amarelo e ter uma indicação visual de que é um elemento clicável. -
A segunda sugestão é utilizar a tag
main.containerao invés dediv.containere tambémh1para o título principal ao invés doh2. Essas mudanças não vão gerar nenhuma mudança visual no seu projeto, mas tornam o seu código HTML mais semântico.
Sobre as tags <h1> a <h6>:
- São usadas para definir títulos no 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.
Espero que ajude!
Fora esses detalhes, seu projeto ficou excelente!
Marked as helpful
@Guga-tab
Posted
@danielmrz-dev Eita, um brasileiro me respondeu kk, muito obrigado pelo feedback Daniel vou colocar em prática a orientação das tags h1...6.
@danielmrz-dev
Posted
Maior parte da galera é de fora, mas tem um ou dois brasileiros por aí rsrs
Fico feliz que o comentário foi útil 😁 @gugaaxe
