Submitted 9 months ago
results summary components main using css grid and flexbox
@oyesina-paul
Design comparison
SolutionDesign
Solution retrospective
What specific areas of your project would you like help with?
Your feedback is appreciated and will go a long way.
Thanks, Paul.
Community feedback
- @danielmrz-devPosted 9 months ago
Hello @oyesina-paul!
Your solution looks great!
I have a suggestion:
- Since you set grid to the body, but you didn't set the size of the rows, your footer is pushing your card up a little bit. You can prevent that from happening adding this to the body:
grid-template-rows: 90% 1fr;
Then your card will always occupy 90% of the space and the footer will adapt in the remaining space.
I hope it helps!
Other than that, you did an excellent job!
0@oyesina-paulPosted 9 months agoHello @danielmrz-dev
Thanks a lot for your review. I really appreciate.
When i tried setting the
grid-template-rows: 90% 1fr;
on my body. The space at the top of my card centering it dissappeared. What do you think caused that or is it just on my desktop?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