Design comparison
SolutionDesign
Solution retrospective
Hi There!
Please Add your own feedback for the design
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @Programming-School-Pro-Coding!
Your project looks great!
I have one suggestion for you to improve it:
- Since you added this to the body:
display: flex; flex-direction: column; align-items: center; justify-content: center;
, if you also addheight: 100vh;
, your card will be placed in the middle of the page and you won't need to add margins to move the card from the borders.
Took a look at your code and updated it for you, like this:
body { font-family: "Hanken Grotesk", sans-serif; margin: 0; height: 100vh; display: flex; flex-direction: column; align-items: center; justify-content: center; }
I hope it helps!
Other than that, great job!
0@Programming-School-Pro-CodingPosted 11 months ago@danielmrz-dev Hey!
Thank you for your feedback.
Yup, this will center the component.
Thank you for your suggestion.
0 - Since you added this to the body:
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