Submitted almost 3 years ago
Easybank landing page solution (with HTML, CSS, JAVASCRIPT)
@basemosman2
Design comparison
SolutionDesign
Solution retrospective
Any feedback is appreciated!
Community feedback
- @skyv26Posted almost 3 years ago
Hi! To solve you accessibility issues:
- Change your code From
<ul class="social-media"> <img src="images/icon-facebook.svg" alt=""> <img src="images/icon-youtube.svg" alt=""> <img src="images/icon-twitter.svg" alt=""> <img src="images/icon-pinterest.svg" alt=""> <img src="images/icon-instagram.svg" alt=""> </ul>
To
<ul class="social-media"> <li><img src="images/icon-facebook.svg" alt=""></li> <li><img src="images/icon-youtube.svg" alt=""></li> <li><img src="images/icon-twitter.svg" alt=""></li> <li><img src="images/icon-pinterest.svg" alt=""></li> <li><img src="images/icon-instagram.svg" alt=""></li> </ul>
- Add meaningful text in all alt attribute. As you can see you left all alt attribute empty.
Solve above will result in zero report.
Good Luck
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