Skip to content
Submitted over 1 year ago

Stats-preview-card (HTML & CSS)

Soraya•230
@Sorpanda
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Stats preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Helpful tips are always welcome. I was wondering if I am maybe declaring things in CSS that are unnecessary or if I am doing things double in different rule sets. I went over it myself and for example I found out that I did a justify content: center, but when I removed it, it didn't change anything at all, thus making it unnecessary. Or I declared a width somewhere and found out I also did it somewhere else, making the first one also unnecessary.

Community feedback

  • Luka Glonti•3,440
    @lack21Posted over 1 year ago

    Great job 👍, but I have a suggestion!

    • Remove margin: 15em 5em from the body and add these styles instead
    {
  display: grid;
  place-content: center;
  min-height: 100vh;
}

    This should center the layout and remove unnecessary space!

    1
    Soraya•230
    @SorpandaPosted over 1 year ago

    Thank you! I had the display grid and place content and it didn't work, I couldn't figure out why. After posting it I saw I forgot the min height I already changed it! @lack21

    1

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