Skip to content
Submitted almost 3 years ago

QR Code Component

João Gabriel•340
@joaojgabriel
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

is this one worth using flex?

Code
Couldn’t fetch repository

Please log in to post a comment

Log in

Community feedback

  • Esteban•150
    @ehmenzala

    Hello João!

    • I think you have done well making this card responsive.
    • Also, semantic HTML is well used.

    Some things you should consider...

    • Maybe you could use max-width: 100%; property on your image to prevent it from stretching when viewport width is being reduced, and remove the height property (Assigning a width to your image will kind of auto-render the height of it, so it will always look good).
    • Also, I think it's ok to use flexbox to center the card. There are various methods to center elements in CSS, and using Flexbox is one of them.

    Good luck! 👋

    Marked as helpful

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