Skip to content

Commit 9b17b9b

Browse files
committed
Use Info's keydown() method to handle ctrl-A (it wasn't working this way).
1 parent 2e5c45b commit 9b17b9b

1 file changed

Lines changed: 9 additions & 9 deletions

File tree

ts/ui/menu/SelectableInfo.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -30,18 +30,18 @@ import { Info, HtmlClasses } from './mj-context-menu.js';
3030
*/
3131
export class SelectableInfo extends Info {
3232
/**
33-
* Add a keypress event to handle "select all" so that only
34-
* the info-box's text is selected (not the whole page)
33+
* Handle "select all" so that only the info-box's text is selected
34+
* (not the whole page)
3535
*
3636
* @override
3737
*/
38-
public addEvents(element: HTMLElement) {
39-
element.addEventListener('keypress', (event: KeyboardEvent) => {
40-
if (event.key === 'a' && (event.ctrlKey || event.metaKey)) {
41-
this.selectAll();
42-
this.stop(event);
43-
}
44-
});
38+
public keydown(event: KeyboardEvent) {
39+
if (event.key === 'a' && (event.ctrlKey || event.metaKey)) {
40+
this.selectAll();
41+
this.stop(event);
42+
return;
43+
}
44+
super.keydown(event);
4545
}
4646

4747
/**

0 commit comments

Comments
 (0)