-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support NPM / Browserify #27
Comments
If you have time for a PR, sure! Could do for a AMD module too! |
Pretty much finished, just waiting for an answer on highcharts/export-csv#72 so the dependency can be expressed correctly in package.json without globals or git references. |
Waiting for highcharts/export-csv#79 |
@A---- Am I understanding correctly that |
Yes :) |
That sucks... I don't really want to add a |
Well,
I had some issues with older versions but on more recent attemps, it worked alright. |
It also patches (global) Highcharts a custom renderer module: https://github.com/highcharts/highcharts-dist/blob/master/modules/canvas-tools.src.js#L2919 I'm currently working to get a NPM package of canvg. PR for export-csv is submitted and jspdf is available on NPM. |
@A---- How are you calling canvg? It must be an XML string, but where does that come from? Do you have a DOM |
The exporting module of Highcharts provides a |
I just found out that Highcharts now supports client side export natively: http://www.highcharts.com/docs/export-module/client-side-export So this module isnt needed anymore. |
Is there any plan to publish on NPM ? Because Highcharts offline export module doesn't support JPEG and PDF yet... |
Would love a npm package for this aswell :) |
It seems like this module is only able to use global Highcharts object. It would be nice if it could be installed with NPM and
require()
d like Highcharts itself.Would submit PR.
The text was updated successfully, but these errors were encountered: