GithubStar/gramps-web
1
0
Fork 0
mirror of https://github.com/gramps-project/gramps-web.git synced 2026-10-16 21:01:38 +08:00
Code Issues Packages Projects Releases Wiki Activity

Esc shouldn't close drawer, only popup

Browse Source
This commit is contained in:
David Straub
2026-10-10 20:32:38 +00:00
parent e1649c45a9
commit c5e7ab96c8
1 changed files with 7 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/components/GrampsjsObjectPreview.js
+7 -3
View File
@@ -167,7 +167,7 @@ export class GrampsjsObjectPreview extends GrampsjsAppStateMixin(LitElement) {
capture: true,
passive: true,
})
window.addEventListener('keydown', this._boundKeyDown)
window.addEventListener('keydown', this._boundKeyDown, true)
}
disconnectedCallback() {
@@ -178,7 +178,7 @@ export class GrampsjsObjectPreview extends GrampsjsAppStateMixin(LitElement) {
window.removeEventListener('db:changed', this._boundDbChanged)
window.removeEventListener('pointerdown', this._boundOutsideInput, true)
window.removeEventListener('wheel', this._boundOutsideInput, true)
window.removeEventListener('keydown', this._boundKeyDown)
window.removeEventListener('keydown', this._boundKeyDown, true)
clearTimeout(this._showTimer)
clearTimeout(this._hideTimer)
}
@@ -259,8 +259,12 @@ export class GrampsjsObjectPreview extends GrampsjsAppStateMixin(LitElement) {
this._hideNow()
}
// Listens in the capture phase so that an Escape that closes a visible popup
// does nothing else, e.g. close the navigation drawer when it has focus.
_handleKeyDown(e) {
if (e.key === 'Escape') this._hideNow()
if (e.key !== 'Escape') return
if (this._visible) e.stopPropagation()
this._hideNow()
}
_position(anchorRect) {