Design comparison
Solution retrospective
Any advice on how to write better CSS is much appreciated (since I'm pretty sure I've kinda' messed some things up, just can't see what...) :)
Community feedback
- @FarisPalayiPosted over 3 years ago
It looks excellent👏 and layout is responding nicely to different screen sizes.
One thing I noticed is that, in mobiles/tablets, the background height is less than the height of the card. It's because its height is 100%. Try to give it a
height
ofauto
andmin-height
of100vh
.And also if you want to center the card. Simply add
display: flex;
andmargin: auto
to body. Or you could add,display: grid
andplace-items: center
. There's a bunch of ways you could do this. I just said some simpler ones.That's all from me. Have fun coding ✨
Marked as helpful1@nyxravenPosted over 3 years ago@FarisPalayi Thank you, didn't even notice that! Will check out the hints and try to fix it. :)
1
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