From cd5081e67ff7bd043441b13db02fd5d268bfa4c0 Mon Sep 17 00:00:00 2001 From: SteVen Batten Date: Wed, 1 Aug 2018 15:20:49 -0700 Subject: [PATCH] fixes #55513 --- src/vs/workbench/browser/parts/menubar/menubarPart.ts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/vs/workbench/browser/parts/menubar/menubarPart.ts b/src/vs/workbench/browser/parts/menubar/menubarPart.ts index 63d53eb8e02..4c1ef5c3267 100644 --- a/src/vs/workbench/browser/parts/menubar/menubarPart.ts +++ b/src/vs/workbench/browser/parts/menubar/menubarPart.ts @@ -656,6 +656,11 @@ export class MenubarPart extends Part { }); this.customMenus[menuIndex].buttonElement.on(EventType.CLICK, (e) => { + // This should only happen for mnemonics and we shouldn't trigger them + if (!this.isVisible) { + return; + } + if (this._modifierKeyStatus && (this._modifierKeyStatus.shiftKey || this._modifierKeyStatus.ctrlKey)) { return; // supress keyboard shortcuts that shouldn't conflict } -- GitLab