QR Code Component

Please log in to post a comment
Log in with GitHubCommunity feedback
- @mjohnson0923
Your solution looks great!
The only suggestion I would make would be to create a separate CSS style sheet to cut down on the code in the HTML file. You can reference the style sheet in the 'head' section and style have it styled the way that you have now.
Keep up the great work!
Marked as helpful - @TaraDesk
Hey, great job on completing the QR Code challenge! 😊
You've done a great job with the card layout and the clean use of flexbox for centering elements—well done
One thing that might help is adding media queries so the card looks better on smaller screens. Right now, it has a fixed size, which could make it a bit squished on mobile. Just tweaking the width and padding would make it way more responsive.
Also, maybe think about using more semantic tags like
<main>
or<section>
. It makes the code cleaner and helps with accessibility too.But honestly, great job—keep it up and happy coding! 😊
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