Responsive component with React and chart.js

Community feedback

• 

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  Hi, Paweł Ciszewski! 👋

  Good job on this challenge! 👏 Your solution looks great and I really like the animation that you added to the bar chart! 🙌

  One thing I'd like to suggest is to specify values such as `margin`, `padding`, `font-size`, and more in a responsive unit such as `em` or `rem` (not `px`, which is a fixed unit), to ensure that changing the font-size of the document causes the rest of the site to scale up/down (which is important for accessibility reasons). To learn more about `em` and `rem`, see this helpful article.

  Hope that helps. 😊

  Keep coding (and happy coding, too)! 😁

  Marked as helpful

  1

  ---

  

  Paweł Ciszewski•340

  @kardahimPosted over 2 years ago

  @ApplePieGiraffe Oh thank you, this is so helpful advice.

  1

  

  T

  ApplePieGiraffe•30,545

  @ApplePieGiraffePosted over 2 years ago

  @kardahim

  Glad to help! 😀

  0