search

mariusmuntean / ChartJs.Blazor

Brings Chart.js charts to Blazor
https://www.iheartblazor.com/
MIT License
677 stars 151 forks source link

Added animation to barchart options. Added duration to ArcAnimations #74

Closed ItsMeDan11 closed 4 years ago

ItsMeDan11 commented 4 years ago

I've added some animation properties in this branch. This is my first attempt at a pull request. Please let me know if I've done things correctly or if changes are required.

zotyovegh commented 4 years ago

Hey! Is this update is also available on the NuGet package? I am trying to reduce the animation speed of a BarChart, but I don't see the extra possibilities you have uploaded with this commit. I have also asked it in a separate question: https://stackoverflow.com/questions/60834550/blazor-reduce-elimination-of-animation-speed-of-a-chart Thank you in advance

Joelius300 commented 4 years ago

As you've seen, I closed this PR. Thank you for the efforts, you absolutely had the right idea. However, it didn't fix all the issues that were present with the animation configuration that's why I decided to open a new issue and fix it directly (a PR was kind of overkill for my single commit). We appreciate the PR and I hope you don't feel discouraged by this. As you can see, the commit that replaced this PR is 10435a4. The issue describing all the problems with the animation configuration is #86.