Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Single price grid component + grid

Carlo 50

@carlobeck

Desktop design screenshot for the Single price grid component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


My second solution Single price grid component using grid.

Please review my code and provide feedback.

Community feedback

gomezlucas 1,115

@gomezlucas

Posted

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

Carlo 50

@carlobeck

Posted

Thanks @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

@fogeid

Posted

It was very cool, remove the border of the button that is perfect!

0

Carlo 50

@carlobeck

Posted

Hi @fogeid, thanks for the feedback. If i remove the bottom border, i would lose the curved corners?

0
Adarsh Pratap 5,515

@adarshcodes

Posted

@carlobeck You can use border-radius property for making corners rounded.

0
Adarsh Pratap 5,515

@adarshcodes

Posted

Hi! @Carlo it'll look nicer if you decrease the width of all the three cards. By the way, good job👍

0

Carlo 50

@carlobeck

Posted

Thanks @adarshcodes for your feedback. I'll try decreasing the width as suggested.

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join 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