Design comparison
Solution retrospective
I would like to hear your feedback! π
Community feedback
- @correlucasPosted about 2 years ago
πΎHi Aman, congrats on completing this challenge!
I've just opened your live site and I can say that you did a great job putting everything together! There's some tips to improve your solution:
The background is not yet correct placed. Add
background-repeat: no-repeat
to avoid the background repeating andbackground-size: contain
to make it fit full width and center with the card this is the best choice, but an alternative to resize it is to usebackground-size: 125%
,body { background: var(--violet) url(../images/bg-desktop.svg); font-family: var(--ff-primary); min-height: 100vh; background-size: contain; background-repeat: no-repeat; }
βοΈ I hope this helps you and happy coding!
Marked as helpful2 - @hyrongennikePosted about 2 years ago
Hi @AmanGupta1703,
Congrats on completing the challenge
You're pretty close to the design just decrease the margin and add a
min-height: 100vh
on the body. That way everything should fit within the viewport.Marked as helpful1
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