Design comparison
Solution retrospective
Hi, folks! I'm unsure I have specific things to ask, but I would love to hear any suggestions from all of you to improve my work. Thank you!
Community feedback
- @denieldenPosted over 2 years ago
Hi Ahmad, great job! Overall you did well 😉
You can remove all
margin
from.card
and allpadding
fromcontainer
classes because with flex they are superfluous and center the card adding flexbox to the bodyHope this help and happy coding!
1@mupinnnPosted over 2 years ago@denielden Hi, Deniel! Thank you so much for the feedback.
I'm thinking the same approach before to use flexbox for centering the card, but not implemented it yet; I think it will hard to adjust the position of the card to match the design when using flexbox, then I realized that easier and can only use
padding-top
(maybe) to adjust the position from the top.I will try it on. Thanks, Deniel. I really appreciate it!
1
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