Thanks @maris.abols, that's a good suggestion. I think Highcharts 2.x and 3.x are actually roughly the same (or close enough to support both). I think there's also a jQuery adapter included with the 2.x version we could use, the 3.x version just includes it by default. Or maybe we shouldn't bother and just tell people to upgrade.
Thanks @kiwad. Fixed. We had previously excluded the other directories because of the dangerous "exporting-server" directory included with highcharts, but now we just have a hook_requirements() informing the user to delete it.
Comments
Comment #1
quicksketchThe D7 version now uses Highcharts and Google Charts (the new ones) out-of-the-box.
Comment #2
maris.abols commentedPlease add to README.txt version of High charts library. I had High charts 2.x version on my demo site and High charts give only error: Object #
has no method 'highcharts'. I suppose High charts works for charts only with 3.x version..Comment #3
quicksketchThanks @maris.abols, that's a good suggestion. I think Highcharts 2.x and 3.x are actually roughly the same (or close enough to support both). I think there's also a jQuery adapter included with the 2.x version we could use, the 3.x version just includes it by default. Or maybe we shouldn't bother and just tell people to upgrade.
Comment #4
kiwad commentedSmall correction to the readme should be done :
sites/all/libraries/highcharts/highcharts.js -> sites/all/libraries/highcharts/js/highcharts.js
Comment #5
quicksketchThanks @kiwad. Fixed. We had previously excluded the other directories because of the dangerous "exporting-server" directory included with highcharts, but now we just have a hook_requirements() informing the user to delete it.