Design comparison
SolutionDesign
Community feedback
- @minimalsmPosted almost 3 years ago
Hey @Mortazamhammed, nice job on this 💪
Few things I'd change:
Important ones
- The '$29 per month' should not be a <h4>, it isn't a heading.
- You've added
padding-left: 10px
to yourh4 span
which is causing the '$29' to also have padding-left.
Nitpicking
- The button should have more height to match the original design
- There is no design guidance for this but "See a hover state on desktop for the Sign Up call-to-action" is one of the stories on this solution
Marked as helpful1@MortazamhammedPosted almost 3 years ago@minimalsm Thank you very much for your frank comments. Your comments helped me
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