Componente de tarjeta de vista previa del producto
Design comparison
Solution retrospective
¿Que te pareció la solución del proyecto? ¿Como optimizarías mas el código?
Community feedback
- @prantiknoorPosted over 2 years ago
Hola @Martincillo94. ¡Felicidades! 🎉🎉. Has completado tu primer desafío.
Quiero darte algunas recomendaciones.
La tarjeta debe estar centrada. Puedes hacerlo fácilmente usando
grid
. Como a continuación:body { background-color: hsl(30deg, 38%, 92%); font-family: "Montserrat", sans-serif; font-size: 0.8rem; color: hsl(228deg, 12%, 48%); min-height: 100vh; display: grid; place-content: center; }
Secondly, you have been given two images for both mobile & desktop. You only used one.
You can use both by using
picture
.<picture class="card__firstSection"> <source srcset="./imagenes/image-product-desktop.jpg" media="(min-width: 600px)" > <img src="./imagenes/image-product-mobile.jpg" alt="imgMobile"> </picture>
En este código, la imagen cambiará de móvil a escritorio cuando el ancho mínimo de la pantalla sea de 600 px o más.
Marked as helpful0@Martincillo94Posted over 2 years ago@prantiknoor es vdd. No me habia dado cuenta de esos detalles.. pronto lo solucionare .gracias por el feedback te lo agradezco mucho 💯🤟 saludos
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