Design comparison
Community feedback
- @rayaattaPosted 10 months ago
hello 👋TheTechHijabie, congratulations on completing this challenge 🎉
I have some suggestions you might find useful.
1 You used
max-width: 370px;
On the.card
This is okay but in case a screen is less than 370px wide then the card overflows the view port. You can make it more responsive by usingmax-width:min(90%,370px)
This means it is 370px but on smaller screens it only covers 90% of the width.2 In order to center the card add
display: grid; place-content: center;
remove
margin:auto;
I hope this helps 🙃.
Your solution looks great 😉
to the css for body.
Marked as helpful0@MajeedatwahabPosted 10 months ago@rayaatta Thank you so much for the feedback, I'll be sure to make the necessary corrections.
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