Skip to content

Commit 0f8a29a

Browse files
Akos Kittakittaakos
Akos Kitta
authored andcommitted
Disabled MenuItem roles on macOS.
Closes #969 Upstream-ref: eclipse-theia/theia#11217 Signed-off-by: Akos Kitta <[email protected]>
1 parent a54d7c8 commit 0f8a29a

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 {
@@ -123,6 +124,15 @@ export class ElectronMainMenuFactory extends TheiaElectronMainMenuFactory {
123124
return { label, submenu };
124125
}
125126

127+
// eslint-disable-next-line @typescript-eslint/no-unused-vars
128+
protected override roleFor(id: string): ElectronMenuItemRole | undefined {
129+
// MenuItem `roles` are completely broken on macOS:
130+
// - https://github.com/eclipse-theia/theia/issues/11217,
131+
// - https://github.com/arduino/arduino-ide/issues/969
132+
// IDE2 uses commands instead.
133+
return undefined;
134+
}
135+
126136
protected override handleElectronDefault(
127137
menuNode: CompositeMenuNode,
128138
args: any[] = [],

0 commit comments

Comments
 (0)