From 117b3eba246ac220abdd43c5a5aad0efa0e2cb9b Mon Sep 17 00:00:00 2001 From: isidor Date: Tue, 5 Apr 2016 19:17:36 +0200 Subject: [PATCH] debug: not all views have setVisible --- src/vs/workbench/parts/debug/browser/debugViewlet.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/vs/workbench/parts/debug/browser/debugViewlet.ts b/src/vs/workbench/parts/debug/browser/debugViewlet.ts index 517451b8c9f2f..8b7cb37c09d10 100644 --- 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 { 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))); }); }