Design comparison
Community feedback
- @RanitManikPosted 5 months ago
Congratulations on your first solution! Here is my feedback for you:
-
Centering the Container: Avoid using
position: absolute
for centering elements. Instead, use a more effective method by applyingmin-height: 100vh;
to thebody
anddisplay: grid; place-items: center;
to center the.container
. This ensures better responsiveness and alignment.body { min-height: 100vh; display: grid; place-items: center; }
Because you used the
position
property, the website overflows on mobile screens as you had to explicitly specify the width and height of the main item. For better reference, you can check out my solution code.
I hope you find these suggestions helpful for improving your site's design and functionality. Feel free to reach out if you have any questions or need further assistance!
Marked as helpful0 -
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