Design comparison
SolutionDesign
Community feedback
- @lack21Posted over 1 year ago
Excellent work 👍, but I have some suggestions!
- Replace
height: 100vh
tomin-height: 100vh
in thebody
, the difference is that, when you setheight: 100vh
to something, that means it won't be bigger than that, it might cause some problems in the future, so better approach is to setmin-height: 100vh
, like this in case content is overflowing container will adjust to it! - Add
margin: 0
to thebody
, this will remove all unnecessary space!
Marked as helpful1@codyivy0Posted over 1 year ago@lack21 Thank you so much for the feedback I very much appreciate it!
1 - Replace
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