Make sure we apply initial style to find widget

mjbvz · mjbvz · commit 615d6408e58d · 2019-10-23T14:16:32.000-07:00
Fixes microsoft#83176
diff --git a/src/vs/workbench/contrib/webview/electron-browser/webviewElement.ts b/src/vs/workbench/contrib/webview/electron-browser/webviewElement.ts
@@ -287,6 +287,8 @@ export class ElectronWebviewBasedWebview extends BaseWebview<WebviewTag> impleme
 			this._register(addDisposableListener(this.element!, 'found-in-page', e => {
 				this._hasFindResult.fire(e.result.matches > 0);
 			}));
+
+			this.styledFindWidget();
 		}
 	}
 
@@ -341,10 +343,11 @@ export class ElectronWebviewBasedWebview extends BaseWebview<WebviewTag> impleme
 
 	protected style(): void {
 		super.style();
+		this.styledFindWidget();
+	}
 
-		if (this._webviewFindWidget) {
-			this._webviewFindWidget.updateTheme(this._webviewThemeDataProvider.getTheme());
-		}
+	private styledFindWidget() {
+		this._webviewFindWidget?.updateTheme(this._webviewThemeDataProvider.getTheme());
 	}
 
 	private readonly _hasFindResult = this._register(new Emitter<boolean>());

Original file line number	Diff line number	Diff line change
`@@ -287,6 +287,8 @@ export class ElectronWebviewBasedWebview extends BaseWebview<WebviewTag> impleme`
`287`	`287`	`this._register(addDisposableListener(this.element!, 'found-in-page', e => {`
`288`	`288`	`this._hasFindResult.fire(e.result.matches > 0);`
`289`	`289`	`}));`
	`290`	`+`
	`291`	`+ this.styledFindWidget();`
`290`	`292`	`}`
`291`	`293`	`}`
`292`	`294`
`@@ -341,10 +343,11 @@ export class ElectronWebviewBasedWebview extends BaseWebview<WebviewTag> impleme`
`341`	`343`
`342`	`344`	`protected style(): void {`
`343`	`345`	`super.style();`
	`346`	`+ this.styledFindWidget();`
	`347`	`+ }`
`344`	`348`
`345`		`- if (this._webviewFindWidget) {`
`346`		`- this._webviewFindWidget.updateTheme(this._webviewThemeDataProvider.getTheme());`
`347`		`- }`
	`349`	`+ private styledFindWidget() {`
	`350`	`+ this._webviewFindWidget?.updateTheme(this._webviewThemeDataProvider.getTheme());`
`348`	`351`	`}`
`349`	`352`
`350`	`353`	`private readonly _hasFindResult = this._register(new Emitter<boolean>());`