mariusmuntean / ChartJs.Blazor

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

Add plugin support #91

Closed Joelius300 closed 4 years ago

Joelius300 commented 4 years ago

Adds the two main properties for plugins.

fixes #63

Currently only the plugin options really make sense, the plugin implementations aren't really helpful when serialized from C#. We could however add a base class for plugins where you can assign methods with the hooks but that would require some additional code in the interop-layer and is only really feasible after #70. That would mean a class that has all the hooks as IMethodHandler properties which are respected on the interop level. This way you could write C# only plugins for chart.js (what madness) but that's an issue for another day.

I have also improved some summaries on the fly just because I was in those classes. In general we don't want to change anything other than what we're actively working on but this is so minor that I thought it makes sense.