Skip to content
Submitted over 2 years ago

Responsive QR Card using css

imagine_lilies•80
@Lilee52
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 didn't face any challenges building this, I hope the replication is similar to the design, kindly drop a feedback on what I could have done better.

Community feedback

  • Kent O'Sullivan•1,870
    @12KentosPosted over 2 years ago

    Hey @Lilee52,

    Project looks great! I looked through your css file and saw you selected a lot of elements directly like so.

    img { max-width: 100%; border-radius: 10px; }

    While this is fine for a small project like this, it Will cause you a Lot of headache in bigger projects. I would suggest selecting your elements either with a class, or with a class and then the element directly.

    Keep up the great work!

    0
    imagine_lilies•80
    @Lilee52Posted over 2 years ago

    @12Kentos thanks, will be mindful of that in future projects.

    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