Eugenia• 950
@JaneMoroz
Posted
Hey! I like your solution! ❤️ The button animation is awesome 🤩
The only things I've noticed:
- If you use button/link without any text (or with only image or icon), I suggest to add
aria-label
to the <button> (or <a>) tag to improve accessibility. You can read more about it here. Also you might want to addaria-hidden
tag for the image inside of the button. More about aria-hidden. For example:
<button aria-label="next advice" class="rounded-full p-5 bg-neonGreen absolute -bottom-[32px]">
<img aria-hidden="true" src="images/icon-dice.svg" alt="">
</button>
- You might want to put
<div class="attribution bg-darkBlue text-lightCyan font-manrope">
inside the <main> tag. It will solve this landmark issue and improve accessibility.
Keep it up! And good luck 🍀
Marked as helpful
1