Design comparison
Solution retrospective
Hi Folks,
I have completed this challenge in less time compared to other challenges and have used CSS Flexbox for the first time. Please review it and let me know for any issues.
Happy coding!
Thanks.
Community feedback
- @abhik-bPosted almost 4 years ago
Hi Ashwini 👋
Fantastic job on this challenge , it is very responsive and it seems perfect
** Just a opinion ** ~ You might wanna center the contents of body for large screens, without margins: (the below way can help you do that)
display: grid; place-items: center; height: 100vh;
Happy coding 😇 & Keep contributing these amazing solutions 🚀
1 - @ApplePieGiraffePosted almost 4 years ago
Greetings, Ashwini! 👋
Well done on this challenge! 👍 Your solution looks good and responds quite nicely! 👏
I only suggest adding a
max-width
to the main container or wrapper to prevent the content of the page from becoming too wide on extra-large screens. 😉Keep coding (and happy coding, too)! 😁
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