Skip to content
OverviewSolutions
2
Comments
3
Bob
@robakerson

All comments

  • Bob•105
    @robakersonPosted over 3 years ago

    Hey Carlos,

    I know this is a small thing but you are missing

    cursor:pointer;

    on your active states for the payment & cancel buttons

    Marked as helpful
    1
  • Albert Nutifafa Dovlo•25
    @alb1nutSubmitted over 3 years ago

    Code Updated. Please help me out by reviewing it. Still not responsive for mobile and web, your opinions on that and any other suggestion will be welcomed. Thank You.

    First solution using CSS Grid

    2
    Bob•105
    @robakersonPosted over 3 years ago

    Few quick changes I would make:

    For the numbers on bottom you used H4, I would use H2 as those numbers are not big enough for you.

    Also, your h4 & p elements have default margins that are pushing your numbers too far from the description. You want "10k+" and "Companies" to be a little closer. I would remove the bottom margin from those H elements and reduce the top margin on the p elements to fix that.

    I would add a "line-height" property to your p elements as well to add space between lines on your middle paragraph. Maybe "line-height:2"?

    Compare the edges of your boxes to the example, you want rounded edges so you are looking for the "border-radius" property. I would experiment with that property to figure out how it works and how you could use it to round the edges of your components.

    Marked as helpful
    1
  • Yohan-Marques•150
    @Yohan-MarquesSubmitted over 3 years ago

    I would like your Feedback, to correct something that is messy or wrong in my code and to have a different point of view to aggregate in my knowledge. Thank you in advance. 🙏🙏🙏

    First responsive design / Flexbox

    2
    Bob•105
    @robakersonPosted over 3 years ago

    You spelled "sign up" wrong

    Marked as helpful
    0