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

Focus search box when pressing slash #131

Closed
wants to merge 2 commits into from
Closed

Conversation

hugovk
Copy link
Member

@hugovk hugovk commented Jun 15, 2023

miketheman added a commit to miketheman/python-docs-theme that referenced this pull request Jul 27, 2023
When the template was made responsive, the choice was made to override
the `body_tag` fragment.
The `body_tag` block is a literal `<body>` which allows the theme developer
to set properties on the `body` tag.
See: sphinx-doc/sphinx#4140

Instead, use the `document` block as documented.
https://www.sphinx-doc.org/en/master/development/templating.html#blocks

Overriding the correct block places the `mobile-nav` section _after_ the
majority of the other HTML parts, which enables the correct function of
the `/` keyboard shortcut.

Resolves python#130
Closes python#131
Closes python#135

Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@hugovk hugovk closed this in #141 Jul 29, 2023
@hugovk hugovk deleted the search-focus branch July 30, 2023 09:43
@hugovk hugovk restored the search-focus branch August 29, 2023 18:42
@hugovk
Copy link
Member Author

hugovk commented Aug 29, 2023

Re-opening as #135 didn't work out.

@hugovk
Copy link
Member Author

hugovk commented Aug 29, 2023

Or the re-open button doesn't work after rebasing/force pushing, so here's a new PR: #153.

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.

Focus search box when pressing slash
2 participants