Design comparison
Solution retrospective
In this Order Summary Component challenge, I've learned how to use flexbox to design the music icon, Annual plan description and change link text in the same row.
Community feedback
- @correlucasPosted about 2 years ago
๐พHello Sudharshan S, Congratulations on completing this challenge!
๐จ Here are some tips to improve your component design:
Your background is applied but its not too similar to the design yet. Add
background-size: contain
instead ofbackground-size: cover
to make it display the size full width and center with the card vertically. Note that now is slightly different from the challenge design.body { background-image: url(images/pattern-background-desktop.svg); background-repeat: no-repeat; background-size: contain; background-color: var(--pale-blue); font-size: 16px; font-family: 'Red Hat Display', sans-serif; }
โ๏ธ I hope this helps you and happy coding!
Marked as helpful1@Sudharshan-24Posted about 2 years ago@correlucas thank you so much for your suggestion Lucas.
1 - @nelsonuprety1Posted about 2 years ago
Looks good. One suggestion i can give you is change the background-image whenever the site is viewed on larger device.
example:
@media (min-width: 768px) { body { /* use your own path for the background desktop svg*/ background: url('./images/pattern-background-desktop.svg') no-repeat; background-color: hsl(225, 100%, 94%); } }
Marked as helpful0@Sudharshan-24Posted about 2 years ago@nelsonuprety1 thank you so much for your suggestion Nelson.
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