Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Documentation fixes #2

Closed
tallsam opened this issue May 6, 2015 · 3 comments
Closed

Documentation fixes #2

tallsam opened this issue May 6, 2015 · 3 comments

Comments

@tallsam
Copy link

tallsam commented May 6, 2015

That bower statement seems incomplete :)

Dependencies are not shipped, so you'll have to do the following:

bower install
@tallsam
Copy link
Author

tallsam commented May 6, 2015

Also typo:

This probably won't work on Internet Explorer has it uses HTML5 download feature and opens a data-uri as a fallback.

@tallsam tallsam changed the title Installation docs missing Documentation fixes May 6, 2015
tallsam pushed a commit to tallsam/highcharts-export-clientside that referenced this issue May 6, 2015
small typo fixes in documentation for issue A----#2
a86e0b2
@A----
Copy link
Owner

A---- commented May 7, 2015

If you do a bower install, it'll fetch dependencies and you'll be able to run example.html. Which should be use mostly for contribution.

Install instructions are written there: http://a----.github.io/highcharts-export-clientside/#installation

But you're right, that should be clarified.

@A----
Copy link
Owner

A---- commented Jul 7, 2015

Sorry to drop your pull request, I (finally) reworked the README page and I don't want to maintain both the README and the website, that's too error-prone.

@A---- A---- closed this as completed Jul 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tallsam @A----