Submitted over 2 years ago
SCSS, JS Vanilla, FETCH API, any advices is welcome as always! Thanks
@RadeQu
Design comparison
SolutionDesign
Community feedback
- @Bayoumi-devPosted over 2 years ago
Hey Radek, It looks great... You just have an
accessibility issue
you need to fix.Buttons must have discernible text
, Don't leave the alt attribute empty.
<button class="changeAdvBtn"> <img src="./images/icon-dice.svg" alt="Advice generator"> </button>
Or Set the attribute
aria-label
to describe this button. like this:<button class="changeAdvBtn" aria-label="Advice generator"> <img src="./images/icon-dice.svg" alt=""> </button>
Keep coding👍
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