Skip to content
Submitted over 1 year ago

QR Code solution

MS2620•60
@MS2620
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

I wrote this code the way I know, if anyone has any suggestions on what they would change, please let me know so I can learn more.

Community feedback

  • visualdennis•8,375
    @visualdennissPosted over 1 year ago

    Very nice job! It looks pretty neat, especially on wide screens.

    On mobile, starting around 550px and below, the body text seems to be a bit squeezed in. My suggestion to avoid that would be avoiding width: 30% to the card, which takes it its width from its parent(body) and therefore equals to 3rd of the screen size, making it narrow on mobile viewports. You could instead set a max-width: 285px or a size you prefer and make the width: 100% instead or something bigger than the 30%. Hope it helps, keep up the good work!

    Marked as helpful
    1

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