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

Add "plotly-basic.min.js" to the project. #212

Closed
IvanJosipovic opened this issue Sep 15, 2022 · 6 comments
Closed

Add "plotly-basic.min.js" to the project. #212

IvanJosipovic opened this issue Sep 15, 2022 · 6 comments
Labels
enhancement New feature or request future work

Comments

@IvanJosipovic
Copy link

Currently, this NuGet package bundles the "plotly-latest.min.js", I would like to use the "plotly-basic.min.js" script as its much smaller in size.

@sean-mcl sean-mcl added enhancement New feature or request future work labels Sep 26, 2022
@tibkiss
Copy link

tibkiss commented Oct 3, 2023

Is it feasible to simply add plotly-latest.min.js (version matched to Plotly.Blazor's built-in version) to wwwroot and replace the javascript import to point to the minimal version?

@sean-mcl
Copy link
Collaborator

sean-mcl commented Oct 3, 2023

Is it feasible to simply add plotly-latest.min.js (version matched to Plotly.Blazor's built-in version) to wwwroot and replace the javascript import to point to the minimal version?

Should be possible I guess

@tibkiss
Copy link

tibkiss commented Oct 3, 2023

@sean-mcl : Thanks for the prompt reply.

The reason I asked, because I tried it out and seemed to work.
Not sure about the internals of Plotly.Blazor which would make this transition unsafe.

Would you consider this hack safe? :)

@sean-mcl
Copy link
Collaborator

sean-mcl commented Oct 3, 2023

@sean-mcl : Thanks for the prompt reply.

The reason I asked, because I tried it out and seemed to work. Not sure about the internals of Plotly.Blazor which would make this transition unsafe.

Would you consider this hack safe? :)

Well, it is safe, as long as of course only what is available in the slimmed down version is used. That's why I don't have any concerns, as long as you know what you're doing. 😄

@tibkiss
Copy link

tibkiss commented Oct 3, 2023

@sean-mcl : Thanks for the prompt reply.
The reason I asked, because I tried it out and seemed to work. Not sure about the internals of Plotly.Blazor which would make this transition unsafe.
Would you consider this hack safe? :)

Well, it is safe, as long as of course only what is available in the slimmed down version is used. That's why I don't have any concerns, as long as you know what you're doing. 😄

Thanks!

@sean-mcl
Copy link
Collaborator

#432 introduces a new property UseBasicVersion to the PlotlyChart-Component.

It will automatically lazy load the basic minified instead of the full version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request future work
Projects
None yet
Development

No branches or pull requests

3 participants