Skip to content
Submitted 5 months ago

Qr-code-component

Gunaseelan•10
@cyborg5818
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

Community feedback

  • Oleg•80
    @scoboxPosted 5 months ago

    Good try overall! I like that you added padding to the card on small resolution to prevent sides of the card sticking to the sides of a screen, i.e having a small gap. Some issues:

    1. Card size is bigger then it is in the design( should be 320px, not 400px).
    2. On small resolutions width/height ratio breaks making the card looking not a per design.
    3. I would not use <body> as flex container. I would suggest using ".container" to have the background color and center the card itself; that solution would require additional <div> with, say, class ".card".
    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