Stats preview card component using HTML and CSS
Design comparison
Solution retrospective
I would really appreciate your feedback!
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hay There ! Karam Good Job on this challenge
These below mentioned tricks will help you remove any Accessibility Issues
-> Add Main tag after body
<main class="container"></main>
-> Learn more on accessibility issues
Keep up the good work!
Marked as helpful1@karamthedevPosted almost 3 years ago@Crazimonk that's quite helpful actually I'm gonna dive deeper into accessibility and semantic HTML best practices. thanks!
1 - @OmarMAttia7Posted almost 3 years ago
Congratulations on finishing the challenge, good job that was solid :).
I have a couple of things to point out:
- You should wrap the main content of your website in a
<main>
element or something equivalent. - You should also get your font-size a little bigger, most people will have trouble reading text this small. In a real-life situation your designer is probably gonna provide you with accurate font sizes but it's good to know anyway.
Good luck and keep coding <3.
Marked as helpful1@karamthedevPosted almost 3 years ago@hazel79 Thanks! I tried to get it as close as possible to the concept design as I'm kinda new to typography and different UI concepts but I would consider looking up that next, I really appreciate your advice.
0 - You should wrap the main content of your website in a
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