Vanza Setia• 27,795
@vanzasetia
Posted
Hi, Amanda P.! 👋
Here are a few ways you can do to improve this solution.
- The illustration and the music icon are decorative images. So, leave the alternative text empty.
- For your information, decorative images are images that don't add any information and serve only aesthetic purposes.
<button>
element must always havetype
attribute to prevent unexpected behavior. Source: Checklist - The A11Y Project #use-the-button-element-for-buttons- Don't change the
<html>
or the:root
font size. It can cause huge accessibility implications for those users with different font sizes or zoom requirements.- Grace Snow explains the issue clearly — Should I change the default HTML font-size to 62.5%?
- Joshua Comeau also does not recommend that approach — The Surprising Truth About Pixels and Accessibility: should I use pixels or rems?
- Remove
<div class="bg__image"></div>
. Then, make the background pattern the background image of the<body>
element.
I hope this helps. Happy coding! 😄
Marked as helpful
2
Amanda P.• 50
@ohkamanda
Posted
Great suggestions. This helps a lot, thank you! @vanzasetia
0
Vanza Setia• 27,795
@vanzasetia
Posted
@ohkamanda
You are welcome! 👍
0