debug: not all views have setVisible

microsoft · Apr 5, 2016 · 117b3eb · 117b3eb
1 parent afb5504
commit 117b3eb
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/vs/workbench/parts/debug/browser/debugViewlet.ts b/src/vs/workbench/parts/debug/browser/debugViewlet.ts
@@ -34,6 +34,7 @@ export class DebugViewlet extends viewlet.Viewlet {
 
 	private $el: builder.Builder;
 	private splitView: splitview.SplitView;
+	// TODO@Isidor views need to be splitView.collapsibleView to make them more general
 	private views: (viewlet.CollapsibleViewletView | viewlet.AdaptiveCollapsibleViewletView)[];
 
 	private lastFocusedView: viewlet.CollapsibleViewletView | viewlet.AdaptiveCollapsibleViewletView;
@@ -89,7 +90,7 @@ export class DebugViewlet extends viewlet.Viewlet {
 
 	public setVisible(visible: boolean): TPromise<any> {
 		return super.setVisible(visible).then(() => {
-			return TPromise.join(this.views.map((view) => view.setVisible(visible)));
+			return TPromise.join(this.views.map((view) => view.setVisible && view.setVisible(visible)));
 		});
 	}