Skip to content
Submitted over 1 year ago

Results summary component

boris2912•270
@boris2912
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Results summary component
  • HTML
  • CSS
1newbie

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

  • Andrés Gutiérrez Ramírez•2,470
    @AGutierrezRPosted over 1 year ago

    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

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