Trigger change event onces in outline provider (#210576)

microsoft · Apr 17, 2024 · debb347 · debb347
1 parent ad220f4
commit debb347
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/vs/workbench/contrib/notebook/browser/viewModel/notebookOutlineProvider.ts b/src/vs/workbench/contrib/notebook/browser/viewModel/notebookOutlineProvider.ts
@@ -253,11 +253,13 @@ export class NotebookCellOutlineProvider {
 			}
 		}));
 
-		this._recomputeActive();
-		this._onDidChange.fire({});
+		const { changeEventTriggered } = this._recomputeActive();
+		if (!changeEventTriggered) {
+			this._onDidChange.fire({});
+		}
 	}
 
-	private _recomputeActive(): void {
+	private _recomputeActive(): { changeEventTriggered: boolean } {
 		let newActive: OutlineEntry | undefined;
 		const notebookEditorWidget = this._editor;
 
@@ -291,7 +293,10 @@ export class NotebookCellOutlineProvider {
 		) {
 			this._activeEntry = newActive;
 			this._onDidChange.fire({ affectOnlyActiveElement: true });
+			return { changeEventTriggered: true };
 		}
+
+		return { changeEventTriggered: false };
 	}
 
 	get isEmpty(): boolean {