Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

NTF Preview Card, plain HTML and CSS

@Carlosgnx

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What did you find difficult while building the project? classnaming haha

Which areas of your code are you unsure of? classnaming and css

please give me feedback about best practices in css and html classnames

Community feedback

Danilo Blas 6,300

@Sdann26

Posted

Hola Carlos, jejeje aquí tambien te tengo unas recomendaciones pero seré más breve. A las imagenes dale siempre que puedas display: block lo que pasa es que por defecto tienen como que un espaciado invisible por eso al ponerle tu overlay queda como si este ocupará más que la imagen, así que le agregas ese atributo y debe quedar perfecto.

Al background que le haz dado al overlay masomenos le haz atinado a lo que se debería hacer cambiale el color por background-color: rgba(0, 255, 247, 0.5) y quedará igual al diseño.

Y ahora si quieres agregar un efecto bien chulo a todo lo que tenga estados activo (hover, focus, active, etc) puedes agregarle la propiedad transition. Esta hará que para pasar de su estado normal al activo tenga una transición haciendo que no sea tan brusco el desplazamiento. Prueba poniendole a los links o botones, transition: all 200ms

Al igual que en el otro proyecto cambia el h2 por h1.

Y si ya haz hecho los cambios que menciono el sujeto de arriba si puedes generá un nuevo reporte llendo a tu reporte para que veas si ya se eliminaron todos los errores de esto,frontend mentor de por si revisa tu index.htmlk de tu repositorio para ver problemas de accesibilidad así que por buenas practicas intentamos que quede sin errores.

Good Coding!

Marked as helpful

1
Abdul 8,540

@Samadeen

Posted

Hey!! Cheers 🥂 on completing this challenge.. .

Here are my suggestions..

  • You should use <main class="card"> instead of <div class="card">.
  • You can wrap your attribution section in a footer tag to avoid accessibility issues.

This should fix most of your accessibility issues

. Regardless you did amazing... hope you find this useful... Happy coding!!!

Marked as helpful

1

@Carlosgnx

Posted

@Samadeen thankyou!

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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