Skip to content
OverviewSolutions
7
Comments
6
Ted Dino
@ted-dino

All comments

  • Ted Dino•170
    @ted-dino

    awesome work!

    Marked as helpful
  • Ted Dino•170
    @ted-dino

    Hi, it look nice but I think you forgot to change the background-color of the "Annual Plan" part and if you like you can fix the accessibility issues.

    Marked as helpful
  • Ted Dino•170
    @ted-dino

    use padding

    Marked as helpful
  • Ted Dino•170
    @ted-dino

    you mean the line height of the paragraph?

    Marked as helpful
  • Ted Dino•170
    @ted-dino

    dang. i guess being a pro account really pay off, ha? it's awesome how you make it almost identical.

  • Ted Dino•170
    @ted-dino

    Hi Shivang, I'm a newbie and also learning here in Frontend Mentor I just want to share what I see on your code. I notice in your styles.css you have a multiple of @media (min-width 50em). I think your css will be more organized if you put them in one @media (min-width 50em) other than that it's nice and it also looks good on mobile. Keep up the good work and try to fix the accessibility issues. Have a great day!

    Marked as helpful