Design comparison
Community feedback
- @Sdann26Posted over 2 years ago
Ramiro, felicitaciones por acabar este proyecto!
Por cierto haz dejado texto que ya has usado en el proyecto, lo puedes ver porque esta posicionado abajo de esta manera:
0.041 ETH 3 days left Creation of Jules Wyvern
Para el efecto que el ojo te quede blanco, no uses opacity en el hover, porque este hará que la opacidad de todos los elementos dentro de este les afecte, por eso el ojo te sale traslucido. Más bien puedes usar un fondo semi transparente como por ejemplo
hover:bg-cyan-400/70
este ultimo valor con slash es para generar el efecto de transparencia.Al icono del reloj te falto ponerle alt, podrías ponerle alt="" y aria-hidden="true" al igual podrías hacerlo en el link.
Por ultimo hay dos div's dentro del body, al primero cambia el div por main y al segundo ponle footer. A partir de ahora siempre usa una etiqueta main para poner el contenido principal del proyecto y siempre debe haber uno.
Bueno espero que estos comentarios te sean de ayuda si tienes más dudas puedes consultarmelas.
Good Coding!
Marked as helpful1@Raam4Posted over 2 years ago@Sdann26 Espectacular, muchísimas gracias por el feedback, seguiré tus recomendaciones.
1@Sdann26Posted over 2 years ago@Raam4 No olvides generar un nuevo reporte apenas termines de editar el código para ver si aun queda algún error :D!
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