Vanza Setia• 27,795
@vanzasetia
Posted
Hi, Leonardo!
Here are a few ways you can do to improve this solution.
- The music icon is a decorative image. So, leave the alternative text blank, (
alt=""
). - "Annual Plan" should be
<h2>
. Next time, remember to not skip heading levels. Or in other words, heading levels must always be in order to give structure to a page. - Learn more — How-to: Accessible heading structure - The A11Y Project
- 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. - Don't use
id
selectors for styling. There are two reasons for not using ID’s to style content:- They mess up specificity because they are too high (the most important reason).
- They are unique identifiers. So, they are not reusable on the same page.
- Use single-class selectors whenever possible. This can help you produce a low-specificity stylesheet. As a result, it prevents you from having specificity issues.
I hope my feedback helps.
Marked as helpful
0