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.

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

Readall rename and Js loading default #127

Merged
merged 2 commits into from
Jul 8, 2017

Conversation

JobJob
Copy link
Contributor

@JobJob JobJob commented Jul 8, 2017

  • readall has been renamed to readstring on Julia 0.6
  • Enables setting a default js loading preference. Doesn't change current behaviour but enables having plotly.js javascript file loaded via embed or from cdn by default.

InteractNext will set js_default to :embed (using Requires.jl - code here) which will make it easier to support PlotlyJS in Mux, Blink, and Juno.

JobJob added 2 commits July 8, 2017 14:36
readall has been renamed to readstring
Doesn't change current behaviour but enables having plotly.js javascript file loaded via embed or from cdn. 

InteractNext will set it to :embed by default, which will make it easier to support PlotlyJS in Mux, Blink, and Juno.
JobJob added a commit to JuliaGizmos/InteractNext.jl that referenced this pull request Jul 8, 2017
simplfies plotlyjs support further - requires JuliaPlots/PlotlyJS.jl#127
@sglyon
Copy link
Member

sglyon commented Jul 8, 2017

Looks innocent enough , thanks!

Let me know when WebIO and InteractNext are ready to go!

@sglyon sglyon merged commit 74c34a6 into JuliaPlots:master Jul 8, 2017
@JobJob
Copy link
Contributor Author

JobJob commented Jul 9, 2017

Thanks! Will do :)

@JobJob JobJob deleted the js-loading-default branch July 10, 2017 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants