Design comparison
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
- @lack21Posted over 1 year ago
Great job 👍, but I have a suggestion!
- Remove
margin: 15em 5em
from thebody
and add these styles instead
{ display: grid; place-content: center; min-height: 100vh; }
This should center the layout and remove unnecessary space!
1@SorpandaPosted over 1 year agoThank 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 - Remove
Please log in to post a comment
Log in with GitHubJoin 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