mirror of
https://github.com/EGroupware/egroupware.git
synced 2024-12-22 14:41:29 +01:00
Fix Shoelace context menu did not properly hide on Esc / click out
This commit is contained in:
parent
f809a5a7a6
commit
5fed37087b
@ -41,16 +41,24 @@ export class EgwMenuShoelace extends LitElement
|
||||
{
|
||||
super();
|
||||
this.structure = _structure;
|
||||
|
||||
this.handleDocumentClick = this.handleDocumentClick.bind(this);
|
||||
this.handleKeypress = this.handleKeypress.bind(this);
|
||||
}
|
||||
|
||||
connectedCallback()
|
||||
{
|
||||
super.connectedCallback();
|
||||
|
||||
document.addEventListener("click", this.handleDocumentClick);
|
||||
document.addEventListener("keydown", this.handleKeypress);
|
||||
}
|
||||
|
||||
disconnectedCallback()
|
||||
{
|
||||
super.disconnectedCallback();
|
||||
document.removeEventListener("click", this.handleDocumentClick);
|
||||
document.removeEventListener("keydown", this.handleKeypress);
|
||||
if(this.popup)
|
||||
{
|
||||
this.popup.remove();
|
||||
@ -102,7 +110,10 @@ export class EgwMenuShoelace extends LitElement
|
||||
|
||||
public hide()
|
||||
{
|
||||
this.popup.active = false;
|
||||
if(this.popup)
|
||||
{
|
||||
this.popup.active = false;
|
||||
}
|
||||
|
||||
// egw_menu always creates a new menu
|
||||
this.remove();
|
||||
@ -131,6 +142,24 @@ export class EgwMenuShoelace extends LitElement
|
||||
}
|
||||
}
|
||||
|
||||
handleDocumentClick(event)
|
||||
{
|
||||
if(!event.composedPath().includes(this))
|
||||
{
|
||||
this.hide();
|
||||
}
|
||||
}
|
||||
|
||||
handleKeypress(event : KeyboardEvent)
|
||||
{
|
||||
if(event.key == "Escape")
|
||||
{
|
||||
event.preventDefault();
|
||||
event.stopPropagation();
|
||||
this.hide();
|
||||
}
|
||||
}
|
||||
|
||||
private itemTemplate(item : egwMenuItem)
|
||||
{
|
||||
if(item.caption == "-")
|
||||
|
Loading…
Reference in New Issue
Block a user