Expenses chart component using fetch and chartjs

Community feedback

• 

  romila•3,570

  @romila2003Posted about 2 years ago

  Hi Tien,

  Congratulations for 🎉 for completing another challenge, the Chart component looks great, and I like that you used Chart.js, I used it too. There are some issues/suggestions want to address:

  1. Rather than using the margin properties to center the card, you can use the flex property instead e.g.

  body {
    display: flex;
    align-items: center;
    justify-content: center;
    min-height: 100vh;
  }
  

  2. You can give the border-radius to each corner of the bars through using this code borderSkipped: false,
  3. When a user hovers over the bars, the color of the bars should give it a lighter opacity which can be done through using the hoverBackgroundColor property. This can be placed under the backgroundColor property.

  Overall, great work and wish you the best for your future projects so keep coding 👍.

  Marked as helpful

  0