Design comparison
Community feedback
- @correlucasPosted over 2 years ago
πΎHello karraras, congratulations for your new solution!
I saw that you've used
overflow: hidden
in the card but this is cropping the container info and making the stats text out of the container. To fix this behavior you need to givemax-width
to make the stats section responsive andflex-wrap
to break the content.You've missed the
border
around the photo:.contant .img img { border-radius: 50%; border: 4px solid white; }
Your background circles are not properly aligned, I did some fixes for your:
body { background-image: url(./images/bg-pattern-top.svg), url(./images/bg-pattern-bottom.svg); background-size: contain, contain; width: 100%; height: 100vh; background-repeat: no-repeat; background-attachment: fixed, fixed; background-position: top -353px left -136px, bottom -434px right -82px; background-color: var(--Dark-cyan); }
π I hope this helps you and happy coding!
0@karrarasPosted over 2 years ago@correlucas first of all I can't thank you enough ,second I have questions where I should gave max-width is to parents or child
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