Remove 'Enter' KeyListener on Dialog which extends AbstractDialog

#1

Hi,
I have a theia dialog component which extends AbstractDialog. I’d like to remove the enter keypress functionality altogether and only use the button onClick event to perform necessary actions.
I see this method in the AbstractDIalog class:
protected addAcceptAction(element: HTMLElement, …additionalEventTypes: K[]): void { this.addKeyListener(element, Key.ENTER, () => this.accept(), …additionalEventTypes); }

How can I remove this key listener? Thanks in advance.

[original thread by rhendricks]

#2

[Hanksha]

I think it comes rather from the DialogOverlayService constructor

 constructor() {
        addKeyListener(document.body, Key.ENTER, e => this.handleEnter(e));
        addKeyListener(document.body, Key.ESCAPE, e => this.handleEscape(e));
    }
#3

[Hanksha]

This will call handleEnter of the current AbstractDialog. So if you override that function and leave the implementation blank you should have the behavior you want.

#4

[rhendricks]

Thanks, I had to override the addAcceptAction method as well as handleEnter to completely eliminate the enter key functionality. Basically, it looks like handleEnter method is invoked when the dialog overlay is in focus and addAcceptAction is triggered when it’s blurred/unfocused. So I had to override both.

#5

[Hanksha]

I would have thought that handleEnter is used for enter key press event from anywhere, regardless of what is focused (since it’s added to the body) and the addAcceptAction is used for when the button has the focus and therefore receives key events, so as long as the button is not focused it won’t react to enter key event.