Design comparison
Solution retrospective
Hello, I have doubts about the responsiveness part, I did it with media query, I think it worked, lol!😅
Community feedback
- @danielmrz-devPosted 11 months ago
Fala Gustavo!
Seu projeto ficou ótimo!
A parte responsiva ficou certinha. Tenho duas sugestões:
-
Adiciona um
hover effect
e umcursor: pointer
no 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.container
ao invés dediv.container
e tambémh1
para 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 helpful1@Guga-tabPosted 11 months ago@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.
1@danielmrz-devPosted 11 months agoMaior parte da galera é de fora, mas tem um ou dois brasileiros por aí rsrs
Fico feliz que o comentário foi útil 😁 @gugaaxe
1 -
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