Mirrors/webamp
1
0
Fork 0
mirror of https://github.com/captbaritone/webamp.git synced 2026-07-22 17:47:16 +00:00
Code Issues Projects Releases Wiki Activity

Clean up PopupMenu a tiny bit

Browse source
This commit is contained in:
Jordan Eldredge 2019-09-21 13:54:53 -07:00
parent 39a6663960
commit d4a16239ed
1 changed files with 18 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

27
modern/src/runtime/PopupMenu.ts
View file

@ -1,6 +1,7 @@
import MakiObject from "./MakiObject";
import { unimplementedWarning } from "../utils";
import { XmlNode } from "../types";
import Group from "./Group";
type Command =
| {
@ -10,19 +11,32 @@ type Command =
disabled: boolean;
}
| {
name: "separator";
id: "separator";
};
class PopupMenu extends MakiObject {
_commands: Command[];
_guiParent: Group;
js_selectCommand: (id: number) => void;
constructor(node: XmlNode, parent: MakiObject, annotations: Object) {
super(node, parent, annotations);
if (!(parent instanceof Group)) {
throw new Error(
"Tried to create a PopupMenu with a parent that is not a GuiObject"
);
}
// MakiOjbect.parent is just a MakiObject, but we expect the parent to have GuiObject properties/methods.
this._guiParent = parent;
this._commands = [];
this.js_selectCommand = (id: number) => {};
}
js_getCommands(): Command[] {
return this._commands;
}
/**
* getclassname()
*
@ -43,12 +57,7 @@ class PopupMenu extends MakiObject {
checked: boolean,
disabled: boolean
): void {
this._commands.push({
name: txt,
id,
checked,
disabled,
});
this._commands.push({ name: txt, id, checked, disabled });
}
addseparator(): void {
@ -60,8 +69,8 @@ class PopupMenu extends MakiObject {
}
popatmouse(): Promise<number> {
this.attributes.x = this.parent.getmouseposx();
this.attributes.y = this.parent.getmouseposy();
this.attributes.x = this._guiParent.getmouseposx();
this.attributes.y = this._guiParent.getmouseposy();
return new Promise(resolve => {
this.js_selectCommand = id => {
this.parent.js_removeChild(this);