Design comparison
Solution retrospective
All feedback is welcome thank you in advance
Community feedback
- @vanzasetiaPosted almost 2 years ago
Hi, @joelsonlins! 👋
I recommend improving the alternative text of the QR code by describing its purpose. In this case, it will navigate the users to the Frontend Mentor website. So, the better alternative text should be "QR code to frontendmentor.io".
Never use
px
unit for font sizes. Userem
orem
instead. Relative units such asrem
andem
can adapt when the users change the browser's font size setting.- rem vs em Units in CSS | DigitalOcean
- The Surprising Truth About Pixels and Accessibility: should I use pixels or rems?
I hope this helps. Happy coding!
0 - @aniket167779Posted almost 2 years ago
Replace <div class="container"> with the main tag and <h2> with <h1> to fix the accessibility issues. click here for more on web-accessibility and semantic html
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