Solution retrospective
I am proud I was able to complete the project and it came out exactly how I wanted it. What I would do differently next time is to make sure my code is as short as possible.
What challenges did you encounter, and how did you overcome them?I encountered some issues when I was using CSS but with a little research, I was able to overcome the challenge.
What specific areas of your project would you like help with?I am looking for other ways in which my codes can be as short as possible.
Please log in to post a comment
Log in with GitHubCommunity feedback
- Account deleted
Your solution looks good, but there are a few things to improve:
The alt text for the QR code should be more descriptive, like alt="QR code linking to Frontend Mentor".
.container has a fixed width of 1396px, which makes it non-responsive. Use max-width and flexbox instead to make it adapt to different screen sizes.
The text color #68778d might have low contrast against the background—worth checking for accessibility.
.innerbox1 isn’t very descriptive; renaming it to .qr-code-wrapper would make the code clearer.
There's an unused .innerbox2{}—removing it would help keep things clean.
Overall, it’s a solid start. With a few tweaks for responsiveness, accessibility, and code clarity, it’ll be even better.
- @Bytehax21
The Font Size of the first para is larger and the width and padding seems to be a little bit different
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