Submitted over 2 years ago
Responsive using purely css and html and vanilla js
@Tomiwayrn
Design comparison
SolutionDesign
Community feedback
- @Cyril-ArinzePosted over 2 years ago
Hello Tomiwa, Kudos, Your design, and functionality are great. However, your HTML has some validation and accessibility issues, notably are:
- no landmark
- no alternate text for images
- no labels for form elements.
To overcome these issues;
- for landmark-related issues: use header tags to indicate the head section of your page, main tags to display the actual content of the page, and footer tags for the footer.
- img tags take alternate text as a fallback or backup in case the image doesn't load due to errors, to achieve this, add ' alt="image-name" ' to your opening image tags. alternate text is also helpful for user who is accessing your webpage using a screenreader
- all form inputs elements should have a label that takes the attribute "for" whose value corresponds with the id of the input element it represents.
if you found my response helpful, do mark them as helpful
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