Skip to content
Submitted 12 months ago

Responsive QRCode Card using Flexbox

Austin Akers•140
@BboyAkers
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

Would love to receive feedback on how should I better structure my Vanilla CSS. Was going to use Sass but decided not to.

Community feedback

  • Melvin Aguilar 🧑🏻‍💻•61,140
    @MelvinAguilarPosted 12 months ago

    Hello there 👋. Good job on completing the challenge !

    My only suggestion is that you use a CSS reset to eliminate the default styles of the browser. Your solution has around 8px of margin on the body element, causing unnecessary scrolling on desktop devices. Additionally, text elements like paragraphs also have margins both above and below.

    I hope you find it useful! 😄

    Happy coding!

    Marked as helpful
    1
    Austin Akers•140
    @BboyAkersPosted 12 months ago

    @MelvinAguilar Thanks for the feedback!!

    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