Abdul Khaliq 🚀• 72,660
@0xabdulkhaliq
Posted
Hello there 👋. Congratulations on successfully completing your first challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
HTML 🏷️:
- This solution generates accessibility error reports due to
non-semanticmarkup
<html>element must have a lang attribute with valid value, so fix it by<html lang="en">
- So fix it by replacing the
<div class="Qr-card">with semantic element<main>to improve accessibility and organization of your page.
- Every site must want at least one
h1element identifying and describing the main content of the page. Anh1heading provides an important navigation point for users of assistive technologies, allowing them to easily find the main content of the page.
- So Every site wants a level-one heading to improve accessibility by reading aloud the heading by screen readers, you can achieve this by adding a
sr-onlyclass to hide it from visual users (it will be useful for visually impaired users)
I hope you find it helpful ! 😄 Above all, the solution you submitted is great
Happy coding!
Marked as helpful
1
