Skip to content
Submitted over 1 year ago

Basic HTML and CSS, using flexbox

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

Basic beginner qr code challenge with html and css.

Community feedback

  • B33th0v3n•70
    @B33th0v3n960Posted over 1 year ago

    nice work!!!

    Just something that needs to change. The body need to be 100vw. I have a 1920x1080 screen and the card is not align to center because the body isn't covering 100 percent of the view port. Secondly, the body has some margin and therefor in small screens the content is push to the left. Make the margin to 0 and use padding instead.

    Als0, just change the div to main for semantic HTML and SEO.

    Keep it up!!!

    Marked as helpful
    1
    SonLuka•60
    @SonLukaPosted over 1 year ago

    @B33th0v3n960 Thank you for the tips, appreciate it.

    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