Design comparison
SolutionDesign
Solution retrospective
Nice project! It provides me the platform to understand grid layout.
Community feedback
- @visualdennissPosted over 1 year ago
Hey there,
nice work! Your submission looks good overall. I'd recommend not using fixed margins to center the component on the whole page. Instead having the container use GRID or FLEXBOX min-height: 100vh; display: grid; place-items: center; is a better solution that will make sure it will always be centered no matter how you resize the browser. You can also use margin: 0 auto; for horizontal centering sometimes.
Hope you find this feedback helpful!
0
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