Skip to content

Lazyload #2454

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

Closed
wants to merge 5 commits into from
Closed

Lazyload #2454

wants to merge 5 commits into from

Conversation

nicolaskruchten
Copy link
Contributor

Supersedes and closes #2260 and closes #1913

pragyagarg642 and others added 5 commits May 9, 2020 13:27
With this change, plotly.js will be loaded on demand. This uses dynamic
imports to achieve this.

Fixes #1913
@nicolaskruchten nicolaskruchten added this to the 4.8.0 milestone May 9, 2020
@nicolaskruchten nicolaskruchten modified the milestones: 4.8.0, 4.9.0 May 14, 2020
@nicolaskruchten nicolaskruchten modified the milestones: 4.9.0, 4.10 Jul 9, 2020
@nicolaskruchten nicolaskruchten modified the milestones: 4.10, v4.x Sep 10, 2020
@pkvprakash
Copy link

I see this fix keeps on moving from milestone to milestone. Waiting for this functionality to be available for our use.

@nicolaskruchten
Copy link
Contributor Author

Yes, I'm sorry about this. The challenge is that this is potentially quite a risky change that could impact thousands of users, and I haven't had the time to fully test it and convince myself that it's a safe change that we're willing to support moving forward. I'm not sure when I'll have the time to do this, to be honest, although I really would like to.

@nicolaskruchten
Copy link
Contributor Author

I do apologize about how this PR was left unattended for so long, but it was superseded by #3142 which was just merged, so the functionality will make it into the next version of the library.

@archmoj archmoj deleted the lazyload branch November 23, 2021 23:34
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.

Lazy load strategy for plotly bundle
3 participants