Submitted over 2 years ago
Responsive QR Code Component using HTML, SCSS, and Webpack
@fidellim
Design comparison
SolutionDesign
Solution retrospective
Feedback are appreciated!
Community feedback
- @Sdann26Posted over 2 years ago
Hi Fidel!
It's perfect, just to avoid the accessibility problem I would change
<h3 class="qr-component-title">Improve your front-end skills by building projects</h3>
to<h1 class="qr-component-title">Improve your front-end skills by building projects</h1>
.If you can add some box-shadow to
<div class="qr-component">
but it's not really necessary but it would give it a special touch.And as I say it looks spectacular, good luck :D!
Marked as helpful1@fidellimPosted over 2 years ago@Sdann26 oh yeah! Thank you for your feedback! Didn't even notice that there is a shadow on the component. :)
1
Please log in to post a comment
Log in with GitHubJoin 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