Design comparison
SolutionDesign
Community feedback
- @kiriasruPosted 4 months ago
Nice!!! Take the following into account:
- You should wrap everything in a semantic tac (<main></main>)
- Change the background-color
- Add some padding (around 17px) to the <div class="qrContainer">
- Add a descriptive alt attribute to <img> tag (alt="QR code" for example). It helps users who rely on screen readers understand the content of the image.
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