Result summary component main

Please log in to post a comment
Log in with GitHubCommunity feedback
- @Code-S1
check out this article https://fedmentor.dev/posts/font-size-px/ to see the right sizing units and your width is too low on the .result you just need to increase it and just give padding on the .radius
- @AdrianoEscarabote
Hi CJuno23, hope you're doing well! I loved how your project turned out, but I’ve got a few suggestions that could be useful:
Since this project is only based on a single page component, there is no need for a h1 tag. It's always a good idea to prevent accessibility errors, so I believe it would be beneficial for you to add a "h1" in this component. Don't worry if you forget about "h1," though; it's a good practice for when you are developing larger sites.
<h1>Welcome to Results summary component</h1>
The rest is fantastic.
Hopefully, you'll find it helpful. 👍
Join 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