Use transaction for collapseAll

gjsjohnmurray · hediet · web-flow · commit 19116d503499 · 2023-11-27T09:15:31.000Z
Co-authored-by: Henning Dieterichs &lt;notify.henning.dieterichs@live.de&gt;
diff --git a/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorViewModel.ts b/src/vs/editor/browser/widget/multiDiffEditorWidget/multiDiffEditorViewModel.ts
@@ -28,9 +28,11 @@ export class MultiDiffEditorViewModel extends Disposable {
 	}
 
 	public collapseAll(): void {
-		for (const d of this.items.get()) {
-			d.collapsed.set(true, undefined);
-		}
+                transaction(tx => {
+		    for (const d of this.items.get()) {
+			    d.collapsed.set(true, tx);
+		    }
+		});
 	}
 
 	constructor(

Original file line number	Diff line number	Diff line change
`@@ -28,9 +28,11 @@ export class MultiDiffEditorViewModel extends Disposable {`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`public collapseAll(): void {`
`31`		`- for (const d of this.items.get()) {`
`32`		`- d.collapsed.set(true, undefined);`
`33`		`- }`
	`31`	`+ transaction(tx => {`
	`32`	`+ for (const d of this.items.get()) {`
	`33`	`+ d.collapsed.set(true, tx);`
	`34`	`+ }`
	`35`	`+ });`
`34`	`36`	`}`
`35`	`37`
`36`	`38`	`constructor(`