Ensures home preview banner is collapsed

Author: d13

src/webviews/apps/home/stateProvider.ts

@@ -65,7 +65,8 @@ export class HomeStateProvider implements Disposable {
 					break;
 
 				case DidChangePreviewEnabled.is(msg):
-					this.state.previewEnabled = msg.params;
+					this.state.previewEnabled = msg.params.previewEnabled;
+					this.state.previewCollapsed = msg.params.previewCollapsed;
 					this.state.timestamp = Date.now();
 
 					this.provider.setValue(this.state, true);

src/webviews/home/homeWebview.ts

@@ -325,15 +325,16 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 		if (isEnabled === undefined) {
 			isEnabled = !this.getPreviewEnabled();
 		}
-		this.container.telemetry.sendEvent('home/preview/toggled', { enabled: isEnabled, version: 'v16' });
-		configuration.updateEffective('home.preview.enabled', isEnabled);
 
-		if (!isEnabled) {
+		if (!this.getPreviewCollapsed()) {
 			this.onCollapseSection({
 				section: 'newHomePreview',
 				collapsed: true,
 			});
 		}
+
+		this.container.telemetry.sendEvent('home/preview/toggled', { enabled: isEnabled, version: 'v16' });
+		configuration.updateEffective('home.preview.enabled', isEnabled);
 	}
 
 	private onCollapseSection(params: CollapseSectionParams) {
@@ -422,7 +423,7 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 	}
 
 	private getPreviewEnabled() {
-		return configuration.get('home.preview.enabled') ?? false;
+		return configuration.get('home.preview.enabled');
 	}
 
 	private getRepositoriesState(): DidChangeRepositoriesParams {
@@ -596,7 +597,10 @@ export class HomeWebviewProvider implements WebviewProvider<State, State, HomeWe
 	}
 
 	private notifyDidChangeConfig() {
-		void this.host.notify(DidChangePreviewEnabled, this.getPreviewEnabled());
+		void this.host.notify(DidChangePreviewEnabled, {
+			previewEnabled: this.getPreviewEnabled(),
+			previewCollapsed: this.getPreviewCollapsed(),
+		});
 	}
 
 	private notifyDidChangeOnboardingIntegration() {

src/webviews/home/protocol.ts

@@ -147,7 +147,14 @@ export const DidCompleteDiscoveringRepositories = new IpcNotification<undefined>
 	'repositories/didCompleteDiscovering',
 );
 
-export const DidChangePreviewEnabled = new IpcNotification<boolean>(scope, 'previewEnabled/didChange');
+export interface DidChangePreviewEnabledParams {
+	previewEnabled: boolean;
+	previewCollapsed: boolean;
+}
+export const DidChangePreviewEnabled = new IpcNotification<DidChangePreviewEnabledParams>(
+	scope,
+	'previewEnabled/didChange',
+);
 
 export const DidChangeRepositoryWip = new IpcNotification<undefined>(scope, 'repository/wip/didChange');