Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Result summary component main

CJuno23 40

@CJuno23

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

Code-S1 150

@Code-S1

Posted

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

1

CJuno23 40

@CJuno23

Posted

@Code-S1 i will. thanks for this. it helps a lot!

0
Adriano 36,930

@AdrianoEscarabote

Posted

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. 👍

1

CJuno23 40

@CJuno23

Posted

@AdrianoEscarabote i will apply this suggestions. thans a lot!

1

Please log in to post a comment

Log in with GitHub
Discord logo

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