Submitted over 1 year ago
Stats Preview Card Component | SCSS, CSS Flexbox
@fortuneiyoha
Design comparison
SolutionDesign
Solution retrospective
Hello, Frontend Mentor Community! 👋
I'm thrilled to announce that I've successfully completed another challenge! 🎉💻
Please check out my solution and provide your valuable feedback.
Thank you for your continued support and encouragement.
Community feedback
- @vanzasetiaPosted over 1 year ago
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 helpful1@fortuneiyohaPosted over 1 year ago@vanzasetia Thanks for your feedback. I'll make the necessary changes.
0 - Only use
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord