Pricing component with toggle w/ js

Community feedback

• 

  David Payne•1,205

  @dpayne713

  Posted over 4 years ago

  Nice job!

  • I like to add the *::before and *::after pseudo elements to your browser resets.

  • I also prefer on the browser reset to add box-sizing: border-box to make the overall box math easier. https://developer.mozilla.org/en-US/docs/Web/CSS/box-sizing

  • Looks like your media query at 850px could come a little sooner ~ 950px as things start to go off screen. OR do another media query that makes cards smaller in-between.

  30
• 

  

  ApplePieGiraffe•30,525

  @ApplePieGiraffe

  Posted over 4 years ago

  Hey, Sandra! 👋

  Great job on this challenge! Your solution looks good and works well! 👏

  One small thing—the cards look a little "jumpy" when the "Learn More" buttons are hovered over because of the 1px border that is being added to them. You should be able to get rid of that quirk by adding the 1px border to the regular state of the button but making it transparent, then changing it's color to white or purple when the button is hovered over. 😉

  Keep coding (and happy coding, too)! 😁

  10