Skip to content
Submitted 29 days ago

QR code component

Roaa-Hussam•20
@Roaa-Hussam
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

What specific areas of your project would you like help with?

I'm looking for an overall review of my code. I’d appreciate feedback on any areas that could be improved, whether it's structure, readability, best practices, or performance. If there's anything I should update or change, please let me know!

Community feedback

  • P
    Ralph Angelo Gonzaga•130
    @codejerooPosted 29 days ago

    The code already looks so clean! I'm also just starting out as a dev, but I think that since this is a small component rather than an entire page, using <section> instead of <main> is more appropriate. The <main> tag is typically reserved for the primary content of a full web page.

    1
    Roaa-Hussam•20
    @Roaa-HussamPosted 27 days ago

    @codejeroo Thank you so much for your feedback! I really appreciate it. That’s a great point about using <section> instead of <main> since this is just a small component. I’ll make that change. Thanks again for your help—I’m still learning too, so I really value any suggestions. Happy coding! 🚀😃

    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