editors - fix mergeAllGroups (#118268) (#118406)

bpasero · web-flow · commit 3a6b7f7cbdc9 · 2021-03-08T11:44:54.000+01:00
diff --git a/src/vs/workbench/browser/parts/editor/editorCommands.ts b/src/vs/workbench/browser/parts/editor/editorCommands.ts
@@ -283,7 +283,7 @@ export function mergeAllGroups(editorGroupService: IEditorGroupsService): void {
 	const target = editorGroupService.activeGroup;
 	for (const group of editorGroupService.getGroups(GroupsOrder.MOST_RECENTLY_ACTIVE)) {
 		if (group === target) {
-			return; // keep target
+			continue; // keep target
 		}
 
 		editorGroupService.mergeGroup(group, target);

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ export function mergeAllGroups(editorGroupService: IEditorGroupsService): void {`
`283`	`283`	`const target = editorGroupService.activeGroup;`
`284`	`284`	`for (const group of editorGroupService.getGroups(GroupsOrder.MOST_RECENTLY_ACTIVE)) {`
`285`	`285`	`if (group === target) {`
`286`		`- return; // keep target`
	`286`	`+ continue; // keep target`
`287`	`287`	`}`
`288`	`288`
`289`	`289`	`editorGroupService.mergeGroup(group, target);`