From 1fc68f3cf274141942b424894645a4f7096ec1f1 Mon Sep 17 00:00:00 2001 From: Christof Marti Date: Thu, 1 Jul 2021 11:17:22 +0200 Subject: [PATCH] Revert "Fix #127621" This reverts commit 07a0575b6cb413785a1e405fe9306d0bbd341403. --- src/vs/workbench/contrib/search/browser/searchView.ts | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/vs/workbench/contrib/search/browser/searchView.ts b/src/vs/workbench/contrib/search/browser/searchView.ts index c5daca6613b..90bb28ccaf0 100644 --- a/src/vs/workbench/contrib/search/browser/searchView.ts +++ b/src/vs/workbench/contrib/search/browser/searchView.ts @@ -510,26 +510,25 @@ export class SearchView extends ViewPane { } refreshTree(event?: IChangeEvent): void { - const setChildrenOpts = { diffIdentityProvider: { getId(element: RenderableMatch) { return element.id; } } }; const collapseResults = this.searchConfig.collapseResults; if (!event || event.added || event.removed) { // Refresh whole tree if (this.searchConfig.sortOrder === SearchSortOrder.Modified) { // Ensure all matches have retrieved their file stat this.retrieveFileStats() - .then(() => this.tree.setChildren(null, this.createResultIterator(collapseResults), setChildrenOpts)); + .then(() => this.tree.setChildren(null, this.createResultIterator(collapseResults))); } else { - this.tree.setChildren(null, this.createResultIterator(collapseResults), setChildrenOpts); + this.tree.setChildren(null, this.createResultIterator(collapseResults)); } } else { // If updated counts affect our search order, re-sort the view. if (this.searchConfig.sortOrder === SearchSortOrder.CountAscending || this.searchConfig.sortOrder === SearchSortOrder.CountDescending) { - this.tree.setChildren(null, this.createResultIterator(collapseResults), setChildrenOpts); + this.tree.setChildren(null, this.createResultIterator(collapseResults)); } else { // FileMatch modified, refresh those elements event.elements.forEach(element => { - this.tree.setChildren(element, this.createIterator(element, collapseResults), setChildrenOpts); + this.tree.setChildren(element, this.createIterator(element, collapseResults)); this.tree.rerender(element); }); }