Vanza Setia• 27,855
@vanzasetia
Posted
Hi, Coty B! Welcome back! 👋
Here are a few suggestions for improvements.
- Wrap the Equilibrium image with an anchor tag. Any element that has interactivity should be wrapped by an interactive element.
- Wrap
<a href="#" class="component-title">Equilibrium #3429</a>with an<h1>. Each page should have one<h1>. - Alternative text for images should not contain any words that are related to "image" (e.g. picture, photo, logo, icon, graphic, avatar, etc). It is already an image element (
<img>) so the screen reader will pronounce it as an image. - Set
box-sizing: border-boxon all elements instead of only on the<body>element using universal selector (*). - Prefer unitless numbers for line-height values to avoid unexpected results. Learn more — line-height - CSS: Cascading Style Sheets | MDN
- Never use
pxunit for font sizes. Useremoreminstead. Relative units such asremandemcan adapt when the users change the browser's font size setting.
I hope this helps. Happy coding! 😄
Marked as helpful
0
@vanzasetia thank you for all the helpful feedback!
0
Vanza Setia• 27,855
@vanzasetia
Posted
@cotybro
You are welcome! 👍
0
