Pricing component

Community feedback

• 

  madhaus•1,040

  @festsnusa

  Posted over 2 years ago

  Hi there! Congratulations for completing this challenge.

  Here's my feedback:

  • to place your main component in the center, you can use grid: display: grid; min-height: 100vh; place-content: center;
  • try to place your styling into the CSS/SCSS file, not inside HTML file;
  • importing google fonts is not a good practice. to download fonts to your hub, you can use "google-webfonts-helper";
  • you can try to use SCSS instead of pure CSS, because you'll be able to use mixins and include nestings;
  • i guess, "Learn More" should have "cursor: pointer";
  • in your JS file, instead of adding styles, you can toggle classes. and instead of using "else", you can use "return".

  Hope you found my feedback useful. Happy coding.

  01

  Marked as helpful