Design comparison
SolutionDesign
Solution retrospective
Any feedback will be highly appreciated.
Thanks in advance๐๐พ
Community feedback
- @correlucasPosted over 2 years ago
๐พHello Lesley, congratulations for your new solution!
Great solution as always, I saw your older solutions and you've great stuff! About this solution you did pretty much everything, the container is beautiful and fully responsive.
The only thing I thhink you can change is the alignment and instead of using margins, do it with flexbox in the
body
.See the changes I did to your code:
body { min-height: 100vh; margin: 0; font-size: 14px; font-family: var(--montserrat); background: var(--cream); display: flex; align-items: center; justify-content: center; flex-direction: column; } } @media (min-width: 700px) .flex { display: flex; } @media (min-width: 700px) .container { width: min(600px, 95%); /* margin: 3rem auto; */ /* margin-top: 15vh; */ /* box-shadow: 5px 3px 20px var(--verydarkblue); */ } .container { width: min(400px, 95%); /* margin: 3rem auto; */ }
๐ I hope this helps you and happy coding!
Marked as helpful0
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