QR code component Solution HTML/SCSS

Please log in to post a comment
Log in with GitHubCommunity feedback
- @MelvinAguilar
Hi @pedrogrl 👋, good job completing this challenge, and welcome to the Frontend Mentor Community! 🎉
I like this solution for the challenge. Here are a few suggestions I've made that you can consider in the future if you're looking to improve the solution further:
- To center the content horizontally in a flexbox layout, use
justify-content: center;
to thebody
. Also, you can remove themargin: 0 auto;
in the .container selector. - Add descriptive text to the
alt
attribute of the images. The text must clearly describe the image. The alt attribute enables screen readers to read the information about on-page images and will be displayed instead if an image file cannot load. - Instead of using pixels in font size, use relative units of measure like
rem
orem
. The font size in absolute length units (px) does not allow users with limited vision to change the text size in some browsers. Reference.
I hope those tips will help you.
Good job, and happy coding!
Marked as helpful - To center the content horizontally in a flexbox layout, use
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