QR code component using HTML, CSS and Flexbox
Design comparison
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
Alt text π·:
-
The
alt
attribute should explain the purpose of the image. Uppon scanning the QR code, the user will be redirected to the frontendmentor.io website, so a betteralt
attribute would beQR code to frontendmentor.io
If you want to learn more about the
alt
attribute, you can read this article. π.
CSS π¨:
- Setting the font-size to 62.5% can attract compatibility issues with third-party libraries or plugins. You can read more about this with this two lectures:
Credit to grace-snow and vanzasetia for pointing this out.
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding!
Marked as helpful2@rostyslav-nazarenkoPosted almost 2 years ago@MelvinAguilar Hi! Thank you so much for the feedback and for providing resources!
Already implemented your suggestions π
The article about
alt
attribute really helped me. I never thought about it that way.I'm not going to set
font-size: 62.5%
from now and will correct the same in the other challenge. I thought that it was a great trick to use. Doing the math without calculator in base of 16 is doable if only using units divided by 4. And here the body text 15px and because of that decided to usefont-size: 62.5%
Thanks for your help. π Have a great day!
0 -
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