Skip to content
Submitted about 2 years ago

QR Code Component

Tony•50
@Snorri967
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 found removing the whitespace from the main element originally to make the design look more or less identical to the preview, but messing with max-width percentages on the main and image element fixed that for me.

Community feedback

  • Melvin Aguilar 🧑🏻‍💻•61,140
    @MelvinAguilarPosted about 2 years ago

    Hi @Snorri967 👋, good job for completing this challenge and welcome to the Frontend Mentor Community! 🎉

    Here are some suggestions you might consider:

    • Use min-height: 100vh to body selector, with this property you set a height and you let the element grow even more if necessary, and remove position: absolute; from the main element.
    • You can directly use max-width: 320px; in your main element and you would not need to implement a media query to modify the width.

    I hope those tips will help you.

    Good Job and happy coding !

    Marked as helpful
    1
    Tony•50
    @Snorri967Posted about 2 years ago

    @MelvinAguilar Thanks for the feedback buddy!

    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