romila• 3,570
@romila2003
Posted
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:
- Rather than using the
margin
properties to center the card, you can use theflex
property instead e.g.
body {
display: flex;
align-items: center;
justify-content: center;
min-height: 100vh;
}
- You can give the border-radius to each corner of the bars through using this code
borderSkipped: false,
- 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 thebackgroundColor
property.
Overall, great work and wish you the best for your future projects so keep coding 👍.
Marked as helpful
0