From 9ccb699fca1add6201ad3ddada095971b6597cd5 Mon Sep 17 00:00:00 2001 From: SimonDanisch Date: Fri, 5 Oct 2018 12:33:13 +0200 Subject: [PATCH] improve comments --- src/Plots.jl | 5 +++-- src/backends/web.jl | 2 +- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Plots.jl b/src/Plots.jl index 6f40334f..448bdfec 100644 --- a/src/Plots.jl +++ b/src/Plots.jl @@ -23,8 +23,9 @@ if isfile(joinpath(@__DIR__, "..", "deps", "deps.jl")) else # This is a bit dirty, but I don't really see why anyone should be forced # to build Plots, while it will just include exactly the below line - # as long as now ENV["PLOTS_HOST_DEPENDENCY_LOCAL"] = "true" is given - # If the above env is set + plotly_local_file_path = "", it will warn in the __init__ functtion + # as long as `ENV["PLOTS_HOST_DEPENDENCY_LOCAL"] = "true"` is not set. + # If the above env is set + `plotly_local_file_path == ""``, + # it will warn in the __init__ function to run build const plotly_local_file_path = "" end diff --git a/src/backends/web.jl b/src/backends/web.jl index 336cdeb0..b1663435 100644 --- a/src/backends/web.jl +++ b/src/backends/web.jl @@ -43,7 +43,7 @@ end function standalone_html_window(plt::AbstractPlot) old = use_local_dependencies[] # save state to restore afterwards - # if we open a browser ourself, we can host local files, so we + # if we open a browser ourself, we can host local files, so # when we have a local plotly downloaded this is the way to go! use_local_dependencies[] = isfile(plotly_local_file_path) filename = write_temp_html(plt)