Dytoma• 570
@Dytoma
Posted
Hey, Good job on completing this challenge however I do have some recommendations for accessibility:
- First I'll suggest you add an
aria-label
attribute to your button as there is no text to describe the button and anaria-hidden="true"
to the image inside the button for screen readers to escape this element.<button id="randomAdvice" aria-label="new advice"><img src="./images/icon-dice.svg" alt="Dice" srcset="" aria-hidden="true"></button>
.
You can read more about ARIA
:
Marked as helpful
1