Design comparison
SolutionDesign
Solution retrospective
any suggestion would be appreciated.
Community feedback
- @EngineerHamzieyPosted over 2 years ago
Hi Emtiaz Ahmed, you have done a great job
I have noticed Your page isn't centered, to fix that, follow the code below
body { /* inorder to center the page vertically and horizontally, you should use flexbox. For that you will need to add min-height 100vh to the body to ensure that it's center vertically on all devices min-height is the best option so that will grow bigger when needed(i.e responsive). min-height NOT HEIGHT, using ordinary height, won't allow responsiveness */ min-height: 100vh; /* then we can now center it with flexbox */ display: flex; flex-direction: column; justify-content: center; align-items: center; }
and I noticed a lot of other errors , so if you don't mind, there are more explanation here on how I did mine after being corrected by mine dearest mentors.
I hope you found this helpful 😊
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