Merge pull request microsoft#78507 from microsoft/joao/fix-78505

roblourens · web-flow · commit 84d62bbbae3b · 2019-08-05T11:01:34.000-07:00
Fix: Settings editor renders all tree elements at once
diff --git a/src/vs/workbench/contrib/preferences/browser/settingsEditor2.ts b/src/vs/workbench/contrib/preferences/browser/settingsEditor2.ts
@@ -281,6 +281,11 @@ export class SettingsEditor2 extends BaseEditor {
 
 	layout(dimension: DOM.Dimension): void {
 		this.dimension = dimension;
+
+		if (!this.isVisible()) {
+			return;
+		}
+
 		this.layoutTrees(dimension);
 
 		const innerWidth = Math.min(1000, dimension.width) - 24 * 2; // 24px padding on left and right;
