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

Fix search.html with newer sphinx versions #34

Merged
merged 1 commit into from Jun 2, 2021

Conversation

mtreinish
Copy link
Member

This commit fixes an issue with search.html when building with newer
versions of sphinx. Since the release of sphinx 3.4.0 the search bar
has not worked when using the qiskit_sphinx_theme package. This is
because we were not loading a required script to make the search bar
work with the new sphinx theme. This was done in the upstream
pytorch_sphinx_theme package with:

pytorch/pytorch_sphinx_theme@d033830

which looks like it was itself ported from the sphinx-rtd-theme package
which fixed this same issue with:

readthedocs/sphinx_rtd_theme#1021

This is just porting that change to qiskit_sphinx_theme which at least
fixes the search bar in local builds for me.

This commit fixes an issue with search.html when building with newer
versions of sphinx. Since the release of sphinx 3.4.0 the search bar
has not worked when using the qiskit_sphinx_theme package. This is
because we were not loading a required script to make the search bar
work with the new sphinx theme. This was done in the upstream
pytorch_sphinx_theme package with:

pytorch/pytorch_sphinx_theme@d033830

which looks like it was itself ported from the sphinx-rtd-theme package
which fixed this same issue with:

readthedocs/sphinx_rtd_theme#1021

This is just porting that change to qiskit_sphinx_theme which at least
fixes the search bar in local builds for me.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Backlog
Development

Successfully merging this pull request may close these issues.

None yet

2 participants