Design comparison
Solution retrospective
I appreciate your comments I hope to continue improving
Greetings :D
Community feedback
- @kacperkwintaPosted almost 3 years ago
Hi 🙋♂️
Looks good (: Almost pixel perfect! 💯
Some tips:
- replace
<h2 class="body-title bg-cyan">Equilibrium #3429</h2>
to<h1>
tag (accessibility) - in
<main>
tag you center all items, but in<body>
tag you don't, and on big screen box is not in the middle
Marked as helpful0@PhynxMoralesPosted almost 3 years ago@kacperkwinta I really appreciate your comments, I have made the adjustments with the <body> and in the H1
Excellent🏆
1 - replace
- @TheLeakyCauldronPosted almost 3 years ago
Hiiiii. This is soooo cool. Like, crazy cool. Everything is so well put together.
Marked as helpful0 - @retrovertigo1981Posted almost 3 years ago
Hola @PhynxMorales!
Excelente trabajo con tu código, felicitaciones, lo único que encontré que puedes mejorar es ajustar un poco la medida del efecto hover en la imagen, ya que por lo menos en mi navegador tiende a salirse un poquito del borde de la imagen, seria lo único que podría mencionar, el resto del código esta muy bien a mi parecer. Felicitaciones nuevamente.
Si gustas puedes revisar mi solución al desafío y comentar al respecto:
https://www.frontendmentor.io/solutions/ntf-card-component-using-html-and-css-Dn9cJV3YP
Saludos desde Chile.
Marked as helpful0@PhynxMoralesPosted almost 3 years ago@retrovertigo1981
En verdad gracias, no había notado que deje el tamaño del hover del mismo que el contenedor padre de la imagen, ya lo puedo corregir gracias por tus comentarios y está increíble como tú lo hiciste muchas felicidades
Agradezco tu apoyo de verdad :D
Saludos desde México.
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