Submitted over 3 years ago
stats-preview-card-component-main Solution(By HTML, CSS)
@chinmayagarwal3007
Design comparison
SolutionDesign
Solution retrospective
All feedbacks are appreciated :)
Community feedback
- @tedikoPosted over 3 years ago
Hello, Tejas! 👋
Congratulations on finishing this challenge! Your solution looks good but there is some room to improve. What I can suggest is:
- Since your
.img
image is decorative your alt text should be provided empty (alt="") so that they can be ignored by assistive technologies, such as screen readers. - Try to work on the class names, they are non-intuitive and hard to read in the code. Instead of
.div-1
,div-2
etc use more specific ones like.intro
,.content
,.stats
etc. Use more descriptive ones. - Read about semantic. Semantic elements lead to more consistent code, they are easier to read and improve accessibility.
Good luck with that, have fun coding! 💪
1@chinmayagarwal3007Posted over 3 years ago@tediko Thank you for your comments, sir; I'll remember your suggestions for the future..
0 - Since your
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