Design comparison
Solution retrospective
Honestamente mi trabajo es este...
Community feedback
- @SoyBenaPosted about 1 year ago
¡Buenas! Le he echado un ojo a tu proyecto y no vas mal encaminado. Hay aspectos en los que podrías trabajar, y que te dejo a continuación:
-
Trata de dar una mayor especificidad al definir las clases de tus divisores. Los nombres deben de aclarar a quien lea el código qué elemento contendrá el divisor y/o para qué sirve, por lo que la sucesión de nombres
container_qr
,container_container_qr
,container_container_container_qr
sería incorrecta, pues no aportas información de qué es lo que hay en cada elemento, ni qué función cumple. -
Aunque es cierto que puedes hacer uso de títulos (
<h1>, <h2>, <h3>, <h4>...
) yo te recomendaría hacer uso de párrafos<p>
. Haciendo uso de estos tendrás un total control de la apariencia del texto, pudiendo amoldarlo tanto como quieras en CSS. -
Al añadir la imagen del QR, especificas la ruta
src="/images/image-qr-code.png"
, sin embargo, en tu repositorio no existe. Es decir, no hay una carpeta de nombreimages
, por lo tanto no puede encontrar el archivoimage-qr-code.png
dentro de esta, y por eso recurre a la descripción alternativa.
Vas por buen camino. Te animo a que consultes mi solución y la del resto de la comunidad para poder trabajar en aquellas cosas que todavía fallan en tu código.
¡Un saludo!
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