Design comparison
SolutionDesign
Community feedback
- @correlucasPosted over 2 years ago
πΎHello Sayam, congratulations for your new solution!
Your solution is already great, the design and all the structure are really good done, something you can consider to fix is the background and here's my tips for that:
To fix the background that's not displaying
full height
you need to increase the body size withmin-height: 100vh;
and give it the proper alignment withdisplay: flex
.See the changes I did to your code below:
body { background-image: url(images/bg-mobile.svg); background-repeat: no-repeat; background-size: contain; min-height: 100vh; background-color: hsl(257, 40%, 49%); display: flex; align-items: center; justify-content: center; }
πΎMy rating for this solution: βββββ
π I hope this helps you and happy coding!
Marked as helpful0@sayamalviPosted over 2 years ago@correlucas Thanks Lucas ! Made the necessary changes π
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