@vanzasetia
Posted
Hi, Fortune O. Iyoha! 👋
Congratulations on finishing this challenge! 🎉
I have a few suggestions for improving your solution:
- Only use
<figure>
when you need to include a<figcaption>
:<figure>
does not need to wrap the<picture>
element. - Decorative image should have empty alternative text: Do not fill the
alt
attribute if you already give anaria-hidden="true"
attribute. - Use list element for the statistic: Do not use the
<h2>
for the statistic numbers. The statistic number and the statistic category should be wrapped by a<span>
or<p>
. The idea is they only make sense if they are read together. For example, "314 templates" does make sense instead of "314" and then "templates".
I hope this helps. Happy coding! 😄
P.S. I just came from LinkedIn and saw your solution on my homepage.
Marked as helpful
@fortuneiyoha
Posted
@vanzasetia Thanks for your feedback. I'll make the necessary changes.
@vanzasetia
Posted
@fortune-i-o You are welcome! 👍