Single Pricing Card Component - HTML & CSS with Flexbox & Grid

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 3 years ago

  Hey there, nicoleanalisecox! 👋

  Nice work on this challenge! 🙌 Your solution looks great! 👍

  I'd like to suggest,

  • Switching to a mobile-friendly layout slightly sooner to avoid the sides of the cards moving off the edges of the screen and therefore not being able to be seen right before the layout changes.
  • Adding cursor: pointer to the button on the card component.

  Keep coding (and happy coding, too)! 😁

  1

  ---

  

  maya•60

  @nicoleanalisecoxPosted over 3 years ago

  @ApplePieGiraffe

  Thank you for your comments! Love the simple addition of the pointer cursor improving the interaction with the button!

  0
• 

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  Hey, just a suggestion. Instead of using max-height as argument in the media query, just use max-width since that is most commonly used and I see that it causes your layout to change because you added width: 311px in the .main-card class. Try removing the width so that even in dev tools, your layout will stay the same and will not change because of the width being written^

  1

  ---

  

  maya•60

  @nicoleanalisecoxPosted over 3 years ago

  Thank you for your feedback!

  0

  

  Raymart Pamplona•16,100

  @pikapikamartPosted over 3 years ago

  @nicoleanalisecox Your welcome ^^

  0