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

Bump plotly.js to 2.32.0 version supporting zindex #3086

Closed
denis-x opened this issue May 16, 2024 · 4 comments · Fixed by #3654
Closed

Bump plotly.js to 2.32.0 version supporting zindex #3086

denis-x opened this issue May 16, 2024 · 4 comments · Fixed by #3654
Labels
enhancement New feature or request
Milestone

Comments

@denis-x
Copy link

denis-x commented May 16, 2024

Description

Hello,

Appreciate the framework, that you are creating. Really enjoy to make my pet project on it. I would like to ask you, if you can bump plotly.js to 2.32.0 version by the next release. plotly.js is currently at version 2.27, the recent version 2.32 has the most wanted feature of having zindex to control relative positions of drawn objects.

BR Denis

@falkoschindler
Copy link
Contributor

Hi @denis-x,
We usually upgrade dependencies during major releases only, because they can introduce breaking changes. Therefore I'll schedule this request for the upcoming 2.0.0 milestone.

@falkoschindler falkoschindler added the enhancement New feature or request label May 17, 2024
@falkoschindler falkoschindler added this to the 2.0.0 milestone May 17, 2024
@denis-x
Copy link
Author

denis-x commented May 29, 2024

Hello,

Good to know, thanks Falko. By the way, is any workaround to overload 2.27 by 2.32 version from static location?
Unfortunately, I did not succeed to achieve it with add_resource or by adding script in HEAD/BODY.
Maybe I did something wrong.

BR Denis

@falkoschindler
Copy link
Contributor

Hi Denis, I found a rather neat workaround to replace the library with a different version. You can simply change the file path the first item of ui.plotly.libraries is pointing to. Assuming you have a custom plotly.min.js in your working directory:

ui.plotly.libraries[0].path = Path('plotly.min.js')

@denis-x
Copy link
Author

denis-x commented May 31, 2024

Herzlichen Dank Falko, ich habe es geschafft!

@falkoschindler falkoschindler linked a pull request Aug 30, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants