mirror of
https://github.com/lensapp/lens.git
synced 2025-05-20 05:10:56 +00:00
Clean up Menu DOM elements
Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
parent
8e8ad6663b
commit
0c325f3b36
@ -93,7 +93,6 @@ export class Menu extends React.Component<MenuProps, State> {
|
|||||||
this.opener.addEventListener(this.props.toggleEvent, this.toggle);
|
this.opener.addEventListener(this.props.toggleEvent, this.toggle);
|
||||||
this.opener.addEventListener("keydown", this.onKeyDown);
|
this.opener.addEventListener("keydown", this.onKeyDown);
|
||||||
}
|
}
|
||||||
this.elem.addEventListener("keydown", this.onKeyDown);
|
|
||||||
window.addEventListener("resize", this.onWindowResize);
|
window.addEventListener("resize", this.onWindowResize);
|
||||||
window.addEventListener("click", this.onClickOutside, true);
|
window.addEventListener("click", this.onClickOutside, true);
|
||||||
window.addEventListener("scroll", this.onScrollOutside, true);
|
window.addEventListener("scroll", this.onScrollOutside, true);
|
||||||
@ -106,7 +105,6 @@ export class Menu extends React.Component<MenuProps, State> {
|
|||||||
this.opener.removeEventListener(this.props.toggleEvent, this.toggle);
|
this.opener.removeEventListener(this.props.toggleEvent, this.toggle);
|
||||||
this.opener.removeEventListener("keydown", this.onKeyDown);
|
this.opener.removeEventListener("keydown", this.onKeyDown);
|
||||||
}
|
}
|
||||||
this.elem.removeEventListener("keydown", this.onKeyDown);
|
|
||||||
window.removeEventListener("resize", this.onWindowResize);
|
window.removeEventListener("resize", this.onWindowResize);
|
||||||
window.removeEventListener("click", this.onClickOutside, true);
|
window.removeEventListener("click", this.onClickOutside, true);
|
||||||
window.removeEventListener("scroll", this.onScrollOutside, true);
|
window.removeEventListener("scroll", this.onScrollOutside, true);
|
||||||
@ -218,7 +216,7 @@ export class Menu extends React.Component<MenuProps, State> {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
onKeyDown(evt: KeyboardEvent) {
|
onKeyDown(evt: React.KeyboardEvent | KeyboardEvent) {
|
||||||
if (!this.isOpen) return;
|
if (!this.isOpen) return;
|
||||||
|
|
||||||
switch (evt.code) {
|
switch (evt.code) {
|
||||||
@ -330,6 +328,7 @@ export class Menu extends React.Component<MenuProps, State> {
|
|||||||
left: this.state?.menuStyle?.left,
|
left: this.state?.menuStyle?.left,
|
||||||
top: this.state?.menuStyle?.top,
|
top: this.state?.menuStyle?.top,
|
||||||
}}
|
}}
|
||||||
|
onKeyDown={this.onKeyDown}
|
||||||
>
|
>
|
||||||
{menuItems}
|
{menuItems}
|
||||||
</ul>
|
</ul>
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user