cleanup

microsoft · Jan 24, 2025 · 7980594 · 7980594
1 parent 119b1d1
commit 7980594
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/vs/code/electron-sandbox/workbench/workbench.ts b/src/vs/code/electron-sandbox/workbench/workbench.ts
@@ -193,7 +193,7 @@
 				}
 			}
 
-			// part: auxiliary sidebar (only when width > 0)
+			// part: auxiliary sidebar
 			if (layoutInfo.auxiliarySideBarWidth > 0) {
 				const auxSideDiv = document.createElement('div');
 				auxSideDiv.style.position = 'absolute';

diff --git a/src/vs/platform/theme/common/themeService.ts b/src/vs/platform/theme/common/themeService.ts
@@ -239,6 +239,6 @@ export interface IPartsSplash {
 
 export interface IPartsSplashWorkspaceOverride {
 	layoutInfo: {
-		auxiliarySideBarWidth: [number, string[] /* workspace ID */];
+		auxiliarySideBarWidth: [number, string[] /* workspace identifier the override applies to */];
 	};
 }
diff --git a/src/vs/platform/theme/electron-main/themeMainService.ts b/src/vs/platform/theme/electron-main/themeMainService.ts
@@ -208,7 +208,7 @@ export class ThemeMainService extends Disposable implements IThemeMainService {
 				}
 			} else {
 				const index = workspaceIds.indexOf(workspace.id);
-				if (index !== -1) {
+				if (index > -1) {
 					workspaceIds.splice(index, 1);
 					changed = true;
 				}
@@ -244,13 +244,13 @@ export class ThemeMainService extends Disposable implements IThemeMainService {
 
 		return {
 			...partSplash,
-			layoutInfo: partSplash.layoutInfo ? {
+			layoutInfo: {
 				...partSplash.layoutInfo,
 				// Only apply an auxiliary bar width when we have a workspace specific
 				// override. Auxiliary bar is not visible by default unless explicitly
 				// opened in a workspace.
 				auxiliarySideBarWidth: typeof auxiliarySideBarWidthOverride === 'number' ? auxiliarySideBarWidthOverride : 0
-			} : undefined
+			}
 		};
 	}