Design comparison
SolutionDesign
Community feedback
- @pperdanaPosted over 1 year ago
Hello there👋!! Congratulations on completing this challange.
- I have some additional recommendations for your code that I think you'll find interesting and valuable.
📌 Image element do not have
alt
attributes or you leave it blankfor example code
<img src="images/image-qr-code.png">
In this code you should add
alt
in your code<img src="images/image-qr-code.png" alt="qr code" >
- This
alt
attribute provides alternative text for images, which is important for accessibility purposes. Screen readers, use the alt attribute to read out loud what the image is about, allowing visually impaired users to understand the content of the page.
I hope you found this helpful! 😊
Happy coding🤖
Marked as helpful0 - @ecemgoPosted over 1 year ago
Some recommendations regarding your code could be of interest to you.
In order to fix the accessibility issues:
- You need to replace
<div class="card">
with the<main class="card">
tag. You'd better use Semantic HTML, and you can also reach more information about it from Using Semantic HTML Tags Correctly. - Each main content needs to start with an h1 element. Your accessibility report states page should contain a level-one heading. So, you should use one
<h1>
element in the<main>
tag. You can replace your<h3>Improve your front-end <br> skills by building projects</h3>
element with the<h1>Improve your front-end <br> skills by building projects</h1>
element.
After committing the changes on GitHub and you need to deploy it as a live site. Finally, you should click generate a new report on this solution page to clear the warnings.
Hope I am helpful. :)
Marked as helpful0 - You need to replace
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