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
Allow custom themes to override embed options query parameter #8021
Conversation
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
51dd8ee
to
764b3c3
Compare
5d989cc
to
9f757b0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't explain how this changed, but the tests fail consistently. It's possible the dark theme was being set at the end of script run which was creating some very subtle change. In that case, updating the screenshot makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
…lit#8021) ## Describe your changes We set the theme based on the query param each time the script run is finished. This can break other theme settings, so we want to respect the query parameters only on initial load (in determining the default theme. ## GitHub Issue Link (if applicable) Closes streamlit#7118 ## Testing Plan - Updated JS Unit Tests for the default theme. There were no tests for the script finished action, so I did not include tests for the removal. --- **Contribution License Agreement** By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.
Describe your changes
We set the theme based on the query param each time the script run is finished. This can break other theme settings, so we want to respect the query parameters only on initial load (in determining the default theme.
GitHub Issue Link (if applicable)
Closes #7118
Testing Plan
Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.