Skip to content
Submitted over 1 year ago

QR Code Component

nxcholas•20
@nxcholas
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

first project i've completed. any suggestions are helpful :)

Code

Please log in to post a comment

Log in

Community feedback

  • Brayden Williams•260
    @redstar504

    Hey, great job!

    One small nitpick, your solution has a scroll bar even though the page should not be scrollable.

    Try this trick, rather than using height: 100vh on the body.

    html {
  height: 100%;
}

body {
  min-height: 100%;
}

    Keep your flex styles on the body, and it should still center your content and eliminate the scrollbar. This is a trick from way back in the day before we had viewport size units. It still works better now for things like this too ;)

    Regards, Brayden

    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