history - fix issue with actions being no-op if window opened

bpasero · bpasero · commit e396583c606d · 2025-11-10T07:59:30.000+01:00
diff --git a/src/vs/workbench/browser/actions/windowActions.ts b/src/vs/workbench/browser/actions/windowActions.ts
@@ -170,7 +170,7 @@ abstract class BaseOpenRecentAction extends Action2 {
 			onDidTriggerItemButton: async context => {
 
 				// Remove
-				if (context.button === this.removeFromRecentlyOpened) {
+				if (context.button === this.removeFromRecentlyOpened || context.button === this.windowOpenedRecentlyOpenedFolder || context.button === this.windowOpenedRecentlyOpenedWorkspace) {
 					await workspacesService.removeRecentlyOpened([context.item.resource]);
 					context.removeItem();
 				}

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ abstract class BaseOpenRecentAction extends Action2 {`
`170`	`170`	`onDidTriggerItemButton: async context => {`
`171`	`171`
`172`	`172`	`// Remove`
`173`		`- if (context.button === this.removeFromRecentlyOpened) {`
	`173`	`+ if (context.button === this.removeFromRecentlyOpened \|\| context.button === this.windowOpenedRecentlyOpenedFolder \|\| context.button === this.windowOpenedRecentlyOpenedWorkspace) {`
`174`	`174`	`await workspacesService.removeRecentlyOpened([context.item.resource]);`
`175`	`175`	`context.removeItem();`
`176`	`176`	`}`