Skip to content
Submitted almost 2 years ago

QR Code Component

Marco•20
@MarcoPonce91
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...QR code component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Buenas noches! Es mi primer desafío en HTML y CSS, la verdad que lo fui codificando y visualizando en la medida que avanzaba con el código. Me costó mucho el poder adaptarlo al tamaño que se requería, a mi parecer y por ser la primera vez en esto, no lo veo tan mal. Quisiera consejos para mejorar y obtener buenas prácticas. Saludos desde Catamarca - Argentina.-

Community feedback

  • Martín•240
    @martinoruePosted almost 2 years ago

    Buenas! Lo veo bárbaro.

    Yo usé flexbox para centrarlo en el body. Si todavía no arrancaste con eso, te recomiendo este recurso para aprenderlo.

    Por otro lado, el warning que te está tirando el reporte de accesibilidad tiene que ver con el uso de un div en lugar de un elemento semántico o, en su defecto, un landmark.

    Suerte!

    Marked as helpful
    0
    Marco•20
    @MarcoPonce91Posted almost 2 years ago

    @martinorue muchas gracias por tu consejo. Para el caso del informe de accesibilidad, debería haber usado un footer?

    0
    Martín•240
    @martinoruePosted almost 2 years ago

    @MarcoPonce91 De nada, espero que te haya sido útil. Con respecto a la pregunta, en algunas soluciones he visto que usan la etiqueta footer en estos casos. Y, según lo que dice acá, entiendo que estaría bien aplicarla en este caso.

    Saludos

    0

Please log in to post a comment

Log in with GitHub

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