Responsive QR code component using HTML and CSS
Design comparison
Community feedback
- @Kmullen444Posted over 2 years ago
Your project looks great! I found out recently that in order for a section tag to be seen as semantically more than a div you also need to have an
aria-labelledby=
that points to a title tag like an<h1></h1>
. You can check out a video that helped me with it here.Since you don't really have a need for an h1 tag in this project, you can create one and use Css to hide it so that it can still be seen by screen readers. You can find an example to the Css class that will do that here.
Hope this helps!
Marked as helpful0 - @AxurynnPosted over 2 years ago
Good job!
To fix accessibility issues, you could add a
main
tag to your HTML file. About the responsiveness, you could addmax-width
properties or usemedia
queries.Overall, you did well! Congratulations! 😊
Marked as helpful0
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