Design comparison
SolutionDesign
Community feedback
- @lcasanPosted about 2 months ago
Hi guy, there are a few points where improvements can be made, especially around semantic HTML:
- Landmark Elements: Including semantic landmarks such as <header>, <main>, and <footer> can significantly improve accessibility. In your case, you can wrap the main content inside a <main> tag:
<main class="container"> </main>
This helps assistive technologies navigate your document better.
- Heading Hierarchy: Ensure that your heading hierarchy is logical. You use <h1> and then directly jump to <h2>, which is fine if the content warrants it, but you might consider adding a more explicit structure to define sections if there are more subsections in the page.
- Image Alt Text: The alt text for the QR code image (alt="QR code") can be more descriptive for accessibility. It’s recommended to explain what the QR code does:
<img class="image-qr-code" src="./assets/image-qr-code.png" alt="Scan this QR code to visit />
Marked as helpful1
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