Skip to content

Commit 946ed5a

Browse files
author
Akos Kitta
committed
Disabled MenuItem roles on macOS.
Closes #969 Upstream-ref: eclipse-theia/theia#11217 Signed-off-by: Akos Kitta <[email protected]>
1 parent 5b486b1 commit 946ed5a

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

Diff for: arduino-ide-extension/src/electron-browser/theia/core/electron-main-menu-factory.ts

+10
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ import {
88
} from '@theia/core/lib/common/menu';
99
import {
1010
ElectronMainMenuFactory as TheiaElectronMainMenuFactory,
11+
ElectronMenuItemRole,
1112
ElectronMenuOptions,
1213
} from '@theia/core/lib/electron-browser/menu/electron-main-menu-factory';
1314
import {
@@ -96,6 +97,15 @@ export class ElectronMainMenuFactory extends TheiaElectronMainMenuFactory {
9697
return { label, submenu };
9798
}
9899

100+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
101+
protected override roleFor(id: string): ElectronMenuItemRole | undefined {
102+
// MenuItem `roles` are completely broken on macOS:
103+
// - https://github.com/eclipse-theia/theia/issues/11217,
104+
// - https://github.com/arduino/arduino-ide/issues/969
105+
// IDE2 uses commands instead.
106+
return undefined;
107+
}
108+
99109
protected handleElectronDefault(
100110
menuNode: CompositeMenuNode,
101111
args: any[] = [],

0 commit comments

Comments
 (0)