Skip to content

Focus search box when pressing slash - #131

Closed
hugovk wants to merge 2 commits into
python:mainfrom
hugovk:search-focus
Closed

Focus search box when pressing slash#131
hugovk wants to merge 2 commits into
python:mainfrom
hugovk:search-focus

Conversation

@hugovk

@hugovkhugovk commented Jun 15, 2023

Copy link
Copy Markdown
Member

Comment threadpython_docs_theme/static/search-focus.js
@mikethemanmiketheman mentioned this pull request Jul 25, 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.
Resolvespython#130Closespython#131Closespython#135
Signed-off-by: Mike Fiedler <miketheman@gmail.com>
@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
Copy Markdown
MemberAuthor

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

@hugovk

Copy link
Copy Markdown
MemberAuthor

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

Sign up for freeto 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

@hugovk@ezio-melotti