Skip to content

Commit 87a18fe

Browse files
author
Akos Kitta
committed
fix: restore menu roles
otherwise cannot copy+paste in editor on macOS? arduino#969 Signed-off-by: Akos Kitta <[email protected]>
1 parent dc91725 commit 87a18fe

File tree

1 file changed

+1
-13
lines changed

1 file changed

+1
-13
lines changed

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

+1-13
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,7 @@ import {
1414
ElectronMenuOptions,
1515
ElectronMainMenuFactory as TheiaElectronMainMenuFactory,
1616
} from '@theia/core/lib/electron-browser/menu/electron-main-menu-factory';
17-
import type {
18-
MenuDto,
19-
MenuRole,
20-
} from '@theia/core/lib/electron-common/electron-api';
17+
import type { MenuDto } from '@theia/core/lib/electron-common/electron-api';
2118
import { inject, injectable } from '@theia/core/shared/inversify';
2219
import {
2320
ArduinoMenus,
@@ -136,15 +133,6 @@ export class ElectronMainMenuFactory extends TheiaElectronMainMenuFactory {
136133
return { label, submenu };
137134
}
138135

139-
// eslint-disable-next-line @typescript-eslint/no-unused-vars, unused-imports/no-unused-vars
140-
protected override roleFor(id: string): MenuRole | undefined {
141-
// MenuItem `roles` are completely broken on macOS:
142-
// - https://github.com/eclipse-theia/theia/issues/11217,
143-
// - https://github.com/arduino/arduino-ide/issues/969
144-
// IDE2 uses commands instead.
145-
return undefined;
146-
}
147-
148136
protected override fillMenuTemplate(
149137
parentItems: MenuDto[],
150138
menuModel: MenuNode,

0 commit comments

Comments
 (0)