Vanza Setia• 27,795
@vanzasetia
Posted
Hi there! 👋
Good effort on this challenge! Your solution looks great! 👏
Some feedback from me.
- Alternative text for images should not be hyphenated (like code). It should be human readable and descriptive. Also, it should not contain any words that related to "image". It's an image element already so the screen readers will pronounce it as an image element.
- The
bg-pattern
is a decorative image so it's best to leave thealt
empty to hide it from the screen reader users. This way, those users can focus on listening to the page content. - Use the name of the person as the alternative text for the profile picture.
- I recommend using
ul
element and wrapping each item withli
rather than usingdiv
. I recommend always writing the HTML markup without any styling (like a document file). This way, you can make a better decision on what HTML element you should use. In this case, if the HTML has no styling then it would make sense to makestats
as a list.- 80K followers
- 803K likes
- 1.4K photos
I hope you find this information beneficial. Happy coding! 😄
Marked as helpful
1
Abubakr Hisham• 320
@abubakr404
Posted
@vanzasetia Hello ! Thank you for this awesome advice.
0
Vanza Setia• 27,795
@vanzasetia
Posted
@abubakr404 You're welcome! 👍
0