#41071 Show the view on selected if hidden

src/vs/workbench/browser/parts/views/viewsViewlet.ts

@@ -324,12 +324,14 @@ export class ViewsViewlet extends PanelViewlet implements IViewsViewlet {
 			.then(() => void 0);
 	}
 
-	focusView(id: string): void {
+	openView(id: string): void {
 		this.focus();
 		const view = this.getView(id);
 		if (view) {
 			view.setExpanded(true);
 			view.focus();
+		} else {
+			this.toggleViewVisibility(id);
 		}
 	}
 
@@ -357,18 +359,16 @@ export class ViewsViewlet extends PanelViewlet implements IViewsViewlet {
 	}
 
 	toggleViewVisibility(id: string): void {
-		const view = this.getView(id);
 		let viewState = this.viewsStates.get(id);
-
 		if (!viewState) {
 			return;
 		}
 
-		viewState.isHidden = !!view;
+		viewState.isHidden = !!this.getView(id);
 		this.updateViews()
 			.then(() => {
 				if (!viewState.isHidden) {
-					this.getView(id).focus();
+					this.openView(id);
 				} else {
 					this.focus();
 				}

src/vs/workbench/common/views.ts

@@ -154,7 +154,7 @@ export const ViewsRegistry: IViewsRegistry = new class implements IViewsRegistry
 
 export interface IViewsViewlet extends IViewlet {
 
-	focusView(id: string): void;
+	openView(id: string): void;
 
 }
 

src/vs/workbench/parts/quickopen/browser/viewPickerHandler.ts

@@ -129,7 +129,7 @@ export class ViewPickerHandler extends QuickOpenHandler {
 			if (views.length) {
 				for (const view of views) {
 					if (this.contextKeyService.contextMatchesRules(view.when)) {
-						result.push(new ViewEntry(view.name, viewlet.name, () => this.viewletService.openViewlet(viewlet.id, true).done(viewlet => (<IViewsViewlet>viewlet).focusView(view.id), errors.onUnexpectedError)));
+						result.push(new ViewEntry(view.name, viewlet.name, () => this.viewletService.openViewlet(viewlet.id, true).done(viewlet => (<IViewsViewlet>viewlet).openView(view.id), errors.onUnexpectedError)));
 					}
 				}
 			}