Design comparison
Community feedback
- @denieldenPosted over 2 years ago
Hi Esteban, I took some time to look at your solution and you did a great job!
Also I have some tips for improving your code:
- remove all
margin
fromcontainer
class - try to use flexbox to the body for center the card. Read here -> best flex guide
- after, add
min-height: 100vh
to body because Flexbox aligns child items to the size of the parent container - try to add a little
transition
on the element with hover effect
Overall you did well :)
Hope this help and happy coding!
Marked as helpful0 - remove all
- @ViicDevPosted over 2 years ago
Buen diseño bro te recomiendo que le des una altura a la card acorde al diseño original, puedes usar VH para darle esa altura, para tener medidas más exactas te recomiendo un medido o usar paint, puedes agregarme a mis redes para poder intercambiar ideas
Marked as helpful0@Geo0510Posted over 2 years ago@PitagoraStrike Gracias por la recomendación bro, justamente estoy viendo eso de como acomodarlo al diseño original.
1@ViicDevPosted over 2 years ago@Geo0510 en lo que pueda ayudar aqui estamos, las medidas yo las calculo con paint pero puedes buscar otras opciones, si gustas pasame un contacto para apoyarnos mutuamente y estoy pensando en hacer directos en Twitch para resolver estos problemas pero cualquier cosa me avisas, suerte...
Marked as helpful0 - Account deleted
Que tal Victor? Mire tu código y esta super bien. quizás falta muy poco de pixel perfecto que es lo que todos buscamos, tienes un excelente comienzo en estos retos. saludos 🙌 , si necesitas algún punto de vista o ayuda en un proyecto send me a messages 🤜🤛
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