Design comparison
SolutionDesign
Community feedback
- @DebabrataBanikPosted about 2 months ago
Your design looks amazing! Few things you might want to look at:
- Remove any default margin and padding from the body and set
min-height: 100vh
. This ensures the layout spans the full viewport height and eliminates unnecessary scrolling. - On your img_container you can add
display: flex
to make sure the image inside is centered and covers the full containers space.
Marked as helpful1@Dzik0Posted about 1 month ago@DebabrataBanik Thank you so much for your feedback, gonna defenietly start using the min-height and gonna remove any default margin/padding!
0 - Remove any default margin and padding from the body and set
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