Skip to content
Submitted 2 months ago

Responsive Card design with QR Code

Shashank Gupta•40
@Shashank23codes
A solution to the QR code component challenge
Preview (opens in new tab)Code (opens in new tab)
Code

Please log in to post a comment

Log in

Community feedback

  • Kama•110
    @Kama-ds10

    Hey Shashank! Awesome work getting this QR Code component up and running 👏 I reviewed your HTML, and you're doing a great job, especially for an early-stage project. Here's a detailed breakdown of what you nailed and what you can improve:

    What You Did Well: Semantic Setup: You've used essential tags like <!DOCTYPE html>, lang="en", meta tags, and included a description. That’s solid for both accessibility and SEO.

    Google Fonts Integration: You included the Outfit font family correctly, which helps match the official design spec.

    Clean Structure: The .card div is a neat container. It keeps everything organized and readable, which is great for maintainability.

    Simple, Readable Code: Your HTML is minimal and clear — no unnecessary elements or messy nesting.

    Suggestions for Improvement: Fix Extra Closing </div>: You have an extra </div> at the bottom of your code:

    </div> <!-- This one doesn't have a matching open tag --> This will break the layout or cause rendering issues. Just delete that line.

    Final Thoughts Powerful work here, Shashank 🙌 With just a few small improvements — like semantic enhancements and better alt text — this will be a super polished beginner project. Keep building and trying new layouts — you’re progressing well!

    Marked as helpful
  • justcallmeea•70
    @justcallmeea
    • I personally struggle to write README documents, and after going through this user's document, it was very enlightening.
    Marked as helpful

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