Result summary component main

Community feedback

• 

  Code-S1•150

  @Code-S1Posted about 1 month ago

  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•50

  @CJuno23Posted about 1 month ago

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

  0
• 

  Adriano•38,250

  @AdrianoEscarabotePosted about 1 month ago

  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•50

  @CJuno23Posted about 1 month ago

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

  1