Design comparison
Solution retrospective
Hello, this is my first task i am a beginner, i will appreciate any feedback
Community feedback
- @bedofaresPosted about 1 year ago
It looks very nice and I like how responsive it is :) . I have only one comment regarding centering it vertically & horizontally. You can achieve this using flexbox .
what you need to change is :
body{ min-height: 100vh; display: flex; align-items: center; justify-content: center; }
and remove the margin-top from the order-summary div.
I hope this helps :)
Marked as helpful1@AhmedAwad97Posted about 1 year ago@bedofares Hello Abdelrahman, I appreciate your feedback. it helps, I have implemented it :)
1 - @DavidIrvine-TWPosted about 1 year ago
hi.. good job :)
you should look at horizontal and vertical centering :)
super easy using flexbox :)
Marked as helpful1@AhmedAwad97Posted about 1 year ago@DavidIrvine-TW thank you for your feedback, I appreciate it. I have modified 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