FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Focus search box when pressing slash by hugovk · Pull Request #131 · python/python-docs-theme · GitHub

Repository navigation

Focus search box when pressing slash - #131

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

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

Conversation

hugovk commented Jun 15, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

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 closed this in #141 Jul 29, 2023
hugovk deleted the search-focus branch July 30, 2023 09:43
hugovk restored the search-focus branch August 29, 2023 18:42

hugovk commented Aug 29, 2023

Copy link
Copy Markdown
Member Author

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

hugovk commented Aug 29, 2023

Copy link
Copy Markdown
Member Author

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL