Design comparison
SolutionDesign
Solution retrospective
Hi, just completed the order summary card challenge but I got some little issues with the background image and the sizing of the card.
Any suggestion would be very welcome !
Community feedback
- @makjnr317Posted about 3 years ago
Hie there.
- You used the wrong link for the background, you should use
/images/pattern-background-mobile.svg
- Also there is no need to add
#main
for everything in your CSS, it just makes coding more boring since you are repeating the same thing. Same as.main__description
and.main__img
, they do not need to be put on. - Instead of using
#main
in your CSS, you can still just saymain
0@abanicaissePosted about 3 years ago@makjnr317 Thank you. I'll try to review my CSS and link the correct background image. Also thanks for the tip about #main it seems to be redundant and I'll try to remove it.
0 - You used the wrong link for the background, you should use
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