-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Stop gobbling escape key #12813
base: master
Are you sure you want to change the base?
Stop gobbling escape key #12813
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not convinced by this - sometimes the document.activeElement
(focus) may be set to other document-level elements (body
, for example) and it seems valid to clear the highlighting in those cases.
This is useful, for example, when searching for a term in some Sphinx-generated HTML documentation, clicking on a result, and then wanting to clear the highlighting without having taking any other action on the page.
@jayaddison, apologies, my wording isn't clear. When I will update the PR description to match this more accurately. That said, you raise a good question. In this PR, links (and |
Fixes #12811.
Subject: restore escape key functionality to HTML modal dialogs in Sphinx
Feature or Bugfix
Purpose
Detail
Relates