Design comparison
Community feedback
- @correlucasPosted over 2 years ago
👾Fala Lucas, beleza? Parabéns pela seu primeiro desafio aqui no FEM.
Acabei de ver o seu live site e está tudo perfeito, dificilmente vejo alguém que acerta todos detalhes logo no primeiro desafio, não tem muito pra falar da solução em si, até a responsividade você acertou!
Meu conselho seria mais sobre fazer um código limpo e sobre semântica, aqui as dicas:
1.Já que você usou
<main>
pra segurar todo o conteúdo do card, não tem necessidade usar a div, tudo que você precisa é <main> e a<img>
,<h1>
e<p>
.2.A estrutura ideal pra essa solução, repetindo o que eu disse antes seria:
<body> <main> <img> <h1></h1> <p></p> </main> </body>
3.Nem tem necessidade de usar classe nessa solução já que chamando cada elemento diretamente no css você já conseguiria selecionar sem problema, visto que só tem uma image, um paragrafo e um título. Então ao invés de usar class, você poderia escrever só
main {}
pra selecionar as propriedades, ao invés de usar.card
.Espero ter te ajudado de qualquer forma e parabéns, ficou bem foda a solução.
0
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