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

Stats-preview-card (HTML & CSS)

Soraya 230

@Sorpanda

Desktop design screenshot for the Stats preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

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

@lack21

Posted

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

@Sorpanda

Posted

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