Skip to content

Commit 1e908ef

Browse files
authored
Secondary side bar shouldnt open by default in vscode.dev (fix microsoft#268390) (microsoft#268606)
1 parent 3855ebd commit 1e908ef

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/vs/workbench/browser/layout.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -632,7 +632,7 @@ export abstract class Layout extends Disposable implements IWorkbenchLayoutServi
632632
private initLayoutState(lifecycleService: ILifecycleService, fileService: IFileService): void {
633633
this._mainContainerDimension = getClientArea(this.parent, this.contextService.getWorkbenchState() === WorkbenchState.EMPTY ? DEFAULT_EMPTY_WINDOW_DIMENSIONS : DEFAULT_WORKSPACE_WINDOW_DIMENSIONS); // running with fallback to ensure no error is thrown (https://github.com/microsoft/vscode/issues/240242)
634634

635-
this.stateModel = new LayoutStateModel(this.storageService, this.configurationService, this.contextService);
635+
this.stateModel = new LayoutStateModel(this.storageService, this.configurationService, this.contextService, this.environmentService);
636636
this.stateModel.load({
637637
mainContainerDimension: this._mainContainerDimension,
638638
resetLayout: Boolean(this.layoutOptions?.resetLayout)
@@ -2789,6 +2789,7 @@ class LayoutStateModel extends Disposable {
27892789
private readonly storageService: IStorageService,
27902790
private readonly configurationService: IConfigurationService,
27912791
private readonly contextService: IWorkspaceContextService,
2792+
private readonly environmentService: IBrowserWorkbenchEnvironmentService,
27922793
) {
27932794
super();
27942795

@@ -2857,6 +2858,10 @@ class LayoutStateModel extends Disposable {
28572858
LayoutStateKeys.SIDEBAR_HIDDEN.defaultValue = workbenchState === WorkbenchState.EMPTY;
28582859
LayoutStateKeys.AUXILIARYBAR_SIZE.defaultValue = Math.min(300, mainContainerDimension.width / 4);
28592860
LayoutStateKeys.AUXILIARYBAR_HIDDEN.defaultValue = (() => {
2861+
if (isWeb && !this.environmentService.remoteAuthority) {
2862+
return true; // TODO@bpasero remove this condition once Chat web support lands
2863+
}
2864+
28602865
const configuration = this.configurationService.inspect(WorkbenchLayoutSettings.AUXILIARYBAR_DEFAULT_VISIBILITY);
28612866

28622867
// Unless auxiliary bar visibility is explicitly configured, make

0 commit comments

Comments
 (0)