Add support for per-element clip attributes#972
Add support for per-element clip attributes#972joshkel wants to merge 1 commit intochartjs:masterfrom
Conversation
|
@joshkel thank you very much! I don't recall why we had some doubts to implement Nevertheless, your PR sounds good even if it would be perfect to move the That was the idea at that time when we talked about that. But it is also clear, my proposal would be a breaking change and therefore it will need a major version. |
|
The purpose of the Regarding whether to put the option in |
Because I mean "moving" and not "adding", because only adding it, you will have the |
|
Nevertheless I think we could proceed step by step, and going on with this implementation and in the new major, change it. |
Fixes #874
I see in #874 that there are potential backwards compatibility concerns. I'm not aware of any in this code as implemented, but I still feel like I don't necessarily understand Chart.js's options implementation, so I may be missing something.
If this feature and approach seems desirable, I'll update tests and docs. Thank you.