Design comparison
Community feedback
- @Illyaas4ShowPosted over 2 years ago
As Salaamu Alaykum, Aya. Welcome to the community!
You should try to:
- Increase the padding of the card.
- Make the border-radius of the QR-code half of the border-radius of the card itself to make it aligned.
- Add
text-align: center;
to the card to make the title and paragraph stay in the center at all screen sizes. - Make the shadow more subtle.
Hope this helped! Salaam! :)
Marked as helpful0@aya880Posted about 2 years ago@Illyaas4Show Thank u ❤️,it really helped I'll take it in consideration
0 - @imadvvPosted over 2 years ago
Greeting Aya Mustafa!! Congratulations on completing Your First challenge!, 👏👏👏 well done.
a few tips that may help you improve your solution, replace
width:21%
on the card tomax-width: 21rem;
, and just remove/* height: 72vh; */ /* margin-top: 3vh; */
to let the card be more responsive on small screens.but over all, Happy Codding, and welcome to Frontend Mentor Community
Marked as helpful0@aya880Posted about 2 years ago@imadbg01 Thank u imad ❤️, and thnx for these useful tips 🌸
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