Skip to content

Commit 84d2ef9

Browse files
authored
TreeView reveal gets stuck if onDidChangeTreeData was fired with empty array (microsoft#211327)
Fixes microsoft#209043
1 parent aaf767a commit 84d2ef9

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/vs/workbench/api/common/extHostTreeViews.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,12 @@ class ExtHostTreeView<T> extends Disposable {
355355
this.dataProvider = options.treeDataProvider;
356356
this.dndController = options.dragAndDropController;
357357
if (this.dataProvider.onDidChangeTreeData) {
358-
this._register(this.dataProvider.onDidChangeTreeData(elementOrElements => this._onDidChangeData.fire({ message: false, element: elementOrElements })));
358+
this._register(this.dataProvider.onDidChangeTreeData(elementOrElements => {
359+
if (Array.isArray(elementOrElements) && elementOrElements.length === 0) {
360+
return;
361+
}
362+
this._onDidChangeData.fire({ message: false, element: elementOrElements });
363+
}));
359364
}
360365

361366
let refreshingPromise: Promise<void> | null;

0 commit comments

Comments
 (0)