
stats-preview-card-component using CSS
Design comparison
Community feedback
- @rayaattaPosted about 1 year ago
Hello @ranikmanit👋 congratulations 🎉 on completing this project I went through your code and here are some tips I think you might find useful. 1.You should always aim at making your html more accessible by using semantic tags learn about semantic tags here Using semantic tags makes your project more findable by search engines, In addition people with disabilities are also able to access and use your site.in some countries accessibility of websites has legal effects so always keep it in mind because divs do not offer any semantic value. 2.in your
* { padding: 0; margin: 0; }
Add
box-sizing: border-box;
. This is recommended as it gives you a more logical way of controlling elements olin your project. When you use this property,the width/height you set for an element is the actual width/height rendered on the screen. 3.it is much more recommended to include the link to your fonts inside your html in order that even when the css has not yet loaded your site still maintains it's original fonts instead of using browser deafaults. And by the way I think the image needs a bit more of color tweaking to look closer to the design I hope you find this helpfulHappy keypressing✌️
Marked as helpful0@RanitManikPosted about 1 year ago@rayaatta Thank you for bringing this to my attention. I will take into consideration all of your suggestions.
1
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