Skip to content
OverviewSolutions
4
Comments
2
Bena
@SoyBena

All comments

  • Bena•40
    @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 nombre images, por lo tanto no puede encontrar el archivo image-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
  • Bena•40
    @SoyBenaSubmitted over 1 year ago

    Hi! As I did some changes it "should" be responsive now. I'd appreciate any comment and/or suggestion ^^

    QR Code Component Task

    1
    Bena•40
    @SoyBenaPosted over 1 year ago

    Hi! Just in case you found this, I've made the following modifications since the original submission:

    • The preview now should be responsive and should adapt to every comercial display.
    • An animation has been added, now you see how the elements kinda slide in!
    0