centered layout: unset margin-left, width, border-color when not centered

isidorn · isidorn · commit fb06298008da · 2018-02-21T13:39:54.000+01:00
diff --git a/src/vs/workbench/browser/parts/editor/editorGroupsControl.ts b/src/vs/workbench/browser/parts/editor/editorGroupsControl.ts
@@ -2211,8 +2211,10 @@ export class EditorGroupsControl extends Themable implements IEditorGroupsContro
 				}
 			}
 
-			editor.getContainer().style({ 'margin-left': `${editorPosition}px`, 'width': `${editorWidth}px` });
-			editor.getContainer().style('border-color', this.getColor(EDITOR_GROUP_BORDER) || this.getColor(contrastBorder));
+			const editorContainer = editor.getContainer();
+			editorContainer.style('margin-left', this.centeredEditorActive ? `${editorPosition}px` : null);
+			editorContainer.style('width', this.centeredEditorActive ? `${editorWidth}px` : null);
+			editorContainer.style('border-color', this.centeredEditorActive ? this.getColor(EDITOR_GROUP_BORDER) || this.getColor(contrastBorder) : null);
 			editor.layout(new Dimension(editorWidth, editorHeight));
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -2211,8 +2211,10 @@ export class EditorGroupsControl extends Themable implements IEditorGroupsContro`
`2211`	`2211`	`}`
`2212`	`2212`	`}`
`2213`	`2213`
`2214`		- editor.getContainer().style({ 'margin-left': `${editorPosition}px`, 'width': `${editorWidth}px` });
`2215`		`- editor.getContainer().style('border-color', this.getColor(EDITOR_GROUP_BORDER) \|\| this.getColor(contrastBorder));`
	`2214`	`+ const editorContainer = editor.getContainer();`
	`2215`	+ editorContainer.style('margin-left', this.centeredEditorActive ? `${editorPosition}px` : null);
	`2216`	+ editorContainer.style('width', this.centeredEditorActive ? `${editorWidth}px` : null);
	`2217`	`+ editorContainer.style('border-color', this.centeredEditorActive ? this.getColor(EDITOR_GROUP_BORDER) \|\| this.getColor(contrastBorder) : null);`
`2216`	`2218`	`editor.layout(new Dimension(editorWidth, editorHeight));`
`2217`	`2219`	`}`
`2218`	`2220`	`}`