Design comparison
Solution retrospective
Hi guys , this is my second project I find it quite okay. Maybe you can tell me what I can do better, I would be happy!
Community feedback
- @FluffyKasPosted about 3 years ago
Hey, your solution looks good and most of the issues were already mentioned so I'd only like to add 2 things:
-
You alt text for "music icon" is probably better just left empty, it's just decorative. You don't need to include words like image, illustration, icon anyway when you're writing alt texts.
-
Your solution would be a bit more responsive if you used
max-width
, instead ofwidth
. There are mobile devices with screens smaller than 450px and your solution unfortunately wouldn't work on those ^^
Marked as helpful2 -
- @CyrusKabirPosted about 3 years ago
hello my dear friend ♥ your card looks good and clean but here some problems :
- the font you use in payment info ( annual plan ) it's not same in design
- cancel order and procced payment button should have hover states
- try to change shadow in button it's not nice and try to close it as possible to design at all it's good but try improve it . good luck ♥♥
Marked as helpful2
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