Design comparison
Solution retrospective
My second solution Single price grid component using grid.
Please review my code and provide feedback.
Community feedback
- @gomezlucasPosted over 4 years ago
Your code looks very clean and readable. I like that. I think it's better to use rem instead of px as a unit. It's an article that help me to understand why. https://medium.com/code-better/css-units-for-font-size-px-em-rem-79f7e592bb97
Basically it makes better the user experience and fewer headaches when you design the responsive layout
1@carlobeckPosted over 4 years agoThanks @gomezlucas for the feedback. I've taken a look at the link you provided, very useful. I'll be use to use rem in my next solution.
0 - @fogeidPosted over 4 years ago
It was very cool, remove the border of the button that is perfect!
0@carlobeckPosted over 4 years agoHi @fogeid, thanks for the feedback. If i remove the bottom border, i would lose the curved corners?
0@adarshcodesPosted over 4 years ago@carlobeck You can use border-radius property for making corners rounded.
0 - @adarshcodesPosted over 4 years ago
Hi! @Carlo it'll look nicer if you decrease the width of all the three cards. By the way, good job👍
0@carlobeckPosted over 4 years agoThanks @adarshcodes for your feedback. I'll try decreasing the width as suggested.
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