don't show title bar context on cmd (microsoft#165557)

sbatten · web-flow · commit 9cf70946b6d5 · 2022-11-04T14:33:25.000-07:00
fixes microsoft#165545
diff --git a/src/vs/workbench/browser/parts/titlebar/titlebarPart.ts b/src/vs/workbench/browser/parts/titlebar/titlebarPart.ts
@@ -285,7 +285,7 @@ export class TitlebarPart extends Part implements ITitleService {
 		// Context menu on title
 		[EventType.CONTEXT_MENU, EventType.MOUSE_DOWN].forEach(event => {
 			this._register(addDisposableListener(this.rootContainer, event, e => {
-				if (e.type === EventType.CONTEXT_MENU || e.metaKey) {
+				if (e.type === EventType.CONTEXT_MENU || (e.target === this.title && e.metaKey)) {
 					EventHelper.stop(e);
 					this.onContextMenu(e, e.target === this.title ? MenuId.TitleBarTitleContext : MenuId.TitleBarContext);
 				}

Original file line number	Diff line number	Diff line change
`@@ -285,7 +285,7 @@ export class TitlebarPart extends Part implements ITitleService {`
`285`	`285`	`// Context menu on title`
`286`	`286`	`[EventType.CONTEXT_MENU, EventType.MOUSE_DOWN].forEach(event => {`
`287`	`287`	`this._register(addDisposableListener(this.rootContainer, event, e => {`
`288`		`- if (e.type === EventType.CONTEXT_MENU \|\| e.metaKey) {`
	`288`	`+ if (e.type === EventType.CONTEXT_MENU \|\| (e.target === this.title && e.metaKey)) {`
`289`	`289`	`EventHelper.stop(e);`
`290`	`290`	`this.onContextMenu(e, e.target === this.title ? MenuId.TitleBarTitleContext : MenuId.TitleBarContext);`
`291`	`291`	`}`