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

Results summary component

boris2912• 270

@boris2912

Desktop design screenshot for the Results summary component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


HI EVERYONE!!

If you have any comments on how I could improve my code I would really appreciate it. let a feedback thaks!!!

Community feedback

@AGutierrezR

Posted

Hello!! It looks great!! I have a few suggestions that might be useful for you:

  1. Change the .container tag into a <main> tag, and this <main> tag should have a min-height: 100vh instead of a height: 800px and of course a justify-content: center; to center the card.
  2. If you set justify-content: center; on the .container you could remove the margin: 20% auto; in the .card.
  3. You could try to use custom properties for the colors, this will give you confidence when using variables in CSS.

Marked as helpful

0

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