search

DuneSt / ChartJs

This is a Seaside binding for ChartJS
MIT License
7 stars 3 forks source link

Move to V2 of ChartJs #1

Open jecisc opened 7 years ago

jecisc commented 7 years ago

ChartJs V2 is out, this project should bind the new version.

I think that the plotbox will need some rework for the V2 version.

svenvc commented 4 years ago

Yes, please, that would be nice.

The differences between https://github.com/chartjs/Chart.js/tree/v1.1.1/docs and the current version is massive.

One point that I would like to add is that doing a full wrapping API is nice, but a lot of work, especially with regards to maintenance.

I would suggest also considering a generic interface that maps more directly to what you learn from the official Chart JS documentation. Maybe both can be combined ?

I found it quite easy to make a simple generic ChartJs2Component based on yours. It is far from complete, but it already works fine for me. I attach a file out for future reference.

ChartJs2Component.st.txt

ThomasBrodt commented 4 years ago

I forked version 1.1.1 and ported it to visualworks 2 years ago. There I am currently on version 2.9.3 of chart.js for the visualworks version. Several plugins were added in the meantime, too, and it is in production use for more than a year now. I am currently updating my fork of the chart.js packages to be in sync with the visualworks version. When I think I am through with it, I will send you a pull request to discuss.