Design comparison
Solution retrospective
Feedback
Community feedback
- @MiculinoPosted almost 3 years ago
Hey @Programing-School, thanks for sharing your solution with us!
I had a look at your final solution and I have a few suggestions to offer that I hope will be useful to you:
-
Change the
background color
to the one in the original design -
Add the two circle background images that can be seen in the original design
-
Don't use
margin
to position your card. There are multiple good ways to do that -flexbox
,CSS grid
,absolute positioning
, etc -
Change the card's
box-shadow
color and reduce its size - make it more subtle -
Make the edges of the card rounded using
border-radius
property -
Center the "London" and "Victor Crest 26" texts
-
Change the card's
background-color
-
Remove the
padding
around the bg pattern image inside the card
Hope this helps. Keep up the great work!
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