Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix dcc.Loading spinner style #2860

Merged
merged 6 commits into from
May 15, 2024
Merged

Conversation

AnnMarieW
Copy link
Collaborator

fixes #2858

Applies overlay_style prop to the dcc.Loading children only and not to the spinner component.

Copy link
Contributor

@T4rk1n T4rk1n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just missing a changelog entry.

@AnnMarieW
Copy link
Collaborator Author

@T4rk1n - Thanks for your speedy review! Yeah, I need to fix the tests too. But I upgraded chrome, and can't run tests locally anymore. Need to figure out how to update the chrome driver - I can't find a version for Linux and Chrome 124.

@T4rk1n
Copy link
Contributor

T4rk1n commented May 14, 2024

@T4rk1n - Thanks for your speedy review! Yeah, I need to fix the tests too. But I upgraded chrome, and can't run tests locally anymore. Need to figure out how to update the chrome driver - I can't find a version for Linux and Chrome 124.

I had the same issue with chrome update, I ended up uninstalling chrome from the ubuntu store and downloaded both chrome and the chromedriver from this page: https://googlechromelabs.github.io/chrome-for-testing/#stable

@AnnMarieW
Copy link
Collaborator Author

@T4rk1n This is ready now 🎉

Copy link
Contributor

@T4rk1n T4rk1n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃

@T4rk1n T4rk1n merged commit 2d21e62 into plotly:dev May 15, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Fix overlay_style in dcc.Loading
2 participants