Submitted about 3 years ago
Frontend Mentor - 3-column preview card component solution
@alessandro-giuzio
Design comparison
SolutionDesign
Solution retrospective
Hello there, please i need fedback
Community feedback
- @darryncodesPosted about 3 years ago
Hi Alessandro,
Really great effort, it looks very close to the design.
A few thoughts from me:
- the
border-radius
property doesn't seem to be in the right place on desktop view - you could add a
max-width
to your.card
class to ensure the card doesn't stretch all the way across the viewport when resizing for mobile (if you addmargin: 0 auto;
too that'll center it nicely) - you could add a hover affect to your buttons to match the design
Good luck with it all!
Marked as helpful2 - the
- @annab6Posted about 3 years ago
Hi Alessandro, I think you need to fix font family on your <p> You could set it up on the body, and then just use Big Shoulders Display for headings. Happy Coding!
Marked as helpful1
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