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

Changes for sphinx-rtd-theme 3.0.0 #51

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

victorlin
Copy link
Member

@victorlin victorlin commented Oct 11, 2024

Description of proposed changes

sphinx-rtd-theme 3.0.0 changed the default behavior to hide this flyout in favor of the Read the Docs flyout which was enabled by default along with the 3.0.0 release. I'll plan to disable the RTD flyout on the RTD dashboard when this commit makes its way into RTD-hosted builds.

Related issue(s)

nextstrain/docs.nextstrain.org#216

Checklist

sphinx-rtd-theme 3.0.0 changed the default behavior to hide this flyout
in favor of the Read the Docs flyout which was enabled by default along
with the 3.0.0 release.¹ I'll plan to disable the RTD flyout on the RTD
dashboard when this commit makes its way into RTD-hosted builds.

¹ https://about.readthedocs.com/blog/2024/07/addons-by-default/
RTD no longer adds its own JS on new builds as of October 7, 2024.¹

¹ https://about.readthedocs.com/blog/2024/07/addons-by-default/
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.

1 participant