Existing trees should react to changes to workbench.list.defaultFindMode (microsoft#156581)

joaomoreno · web-flow · commit f4d4971d75fe · 2022-07-28T16:37:06.000+02:00
fixes microsoft#156249
diff --git a/src/vs/base/browser/ui/tree/abstractTree.ts b/src/vs/base/browser/ui/tree/abstractTree.ts
@@ -876,7 +876,8 @@ class FindController<T, TFilterData> implements IDisposable {
 			return;
 		}
 
-		this.widget = new FindWidget(this.view.getHTMLElement(), this.tree, this.contextViewProvider, this._mode, this.styles);
+		this.mode = this.tree.options.defaultFindMode ?? TreeFindMode.Highlight;
+		this.widget = new FindWidget(this.view.getHTMLElement(), this.tree, this.contextViewProvider, this.mode, this.styles);
 		this.enabledDisposables.add(this.widget);
 
 		this.widget.onDidChangeValue(this.onDidChangeValue, this, this.enabledDisposables);

Original file line number	Diff line number	Diff line change
`@@ -876,7 +876,8 @@ class FindController<T, TFilterData> implements IDisposable {`
`876`	`876`	`return;`
`877`	`877`	`}`
`878`	`878`
`879`		`- this.widget = new FindWidget(this.view.getHTMLElement(), this.tree, this.contextViewProvider, this._mode, this.styles);`
	`879`	`+ this.mode = this.tree.options.defaultFindMode ?? TreeFindMode.Highlight;`
	`880`	`+ this.widget = new FindWidget(this.view.getHTMLElement(), this.tree, this.contextViewProvider, this.mode, this.styles);`
`880`	`881`	`this.enabledDisposables.add(this.widget);`
`881`	`882`
`882`	`883`	`this.widget.onDidChangeValue(this.onDidChangeValue, this, this.enabledDisposables);`