Reveal Webview in active view column.

rebornix · rebornix · commit 617fc4934f1e · 2019-03-29T15:28:46.000-07:00
diff --git a/src/vs/workbench/api/electron-browser/mainThreadWebview.ts b/src/vs/workbench/api/electron-browser/mainThreadWebview.ts
@@ -200,9 +200,9 @@ export class MainThreadWebviews extends Disposable implements MainThreadWebviews
 			return;
 		}
 
-		const targetGroup = this._editorGroupService.getGroup(viewColumnToEditorGroup(this._editorGroupService, showOptions.viewColumn));
+		const targetGroup = this._editorGroupService.getGroup(viewColumnToEditorGroup(this._editorGroupService, showOptions.viewColumn)) || this._editorGroupService.getGroup(webview.group || 0);
 		if (targetGroup) {
-			this._webviewService.revealWebview(webview, targetGroup || this._editorGroupService.getGroup(webview.group || ACTIVE_GROUP), !!showOptions.preserveFocus);
+			this._webviewService.revealWebview(webview, targetGroup, !!showOptions.preserveFocus);
 		}
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -200,9 +200,9 @@ export class MainThreadWebviews extends Disposable implements MainThreadWebviews`
`200`	`200`	`return;`
`201`	`201`	`}`
`202`	`202`
`203`		`- const targetGroup = this._editorGroupService.getGroup(viewColumnToEditorGroup(this._editorGroupService, showOptions.viewColumn));`
	`203`	`+ const targetGroup = this._editorGroupService.getGroup(viewColumnToEditorGroup(this._editorGroupService, showOptions.viewColumn)) \|\| this._editorGroupService.getGroup(webview.group \|\| 0);`
`204`	`204`	`if (targetGroup) {`
`205`		`- this._webviewService.revealWebview(webview, targetGroup \|\| this._editorGroupService.getGroup(webview.group \|\| ACTIVE_GROUP), !!showOptions.preserveFocus);`
	`205`	`+ this._webviewService.revealWebview(webview, targetGroup, !!showOptions.preserveFocus);`
`206`	`206`	`}`
`207`	`207`	`}`
`208`	`208`