Fix: Default sidebar to open on desktop, use route param as override

Author: adamziel

packages/playground/website/src/lib/state/redux/slice-ui.ts

@@ -128,17 +128,20 @@ export interface UIState {
 const query = new URL(document.location.href).searchParams;
 const isEmbeddedInAnIframe = window.self !== window.top;
 // @TODO: Centralize these breakpoint sizes.
-// eslint-disable-next-line @typescript-eslint/no-unused-vars
 const isMobile = window.innerWidth < 875;
 
 // Parse the route parameter for sidebar/tab state
 const routeState = parseRouteParam(query.get('route'));
-
-// Check if there's an explicit route param requesting the sidebar to be open.
-// If so, respect it regardless of viewport size (but not in seamless mode or iframes).
-const hasExplicitRouteParam = query.has('route') && routeState.sidebarOpen;
 const isSeamlessMode = query.get('mode') === 'seamless';
 
+// Determine initial sidebar open state:
+// - If route param exists, use it (route=closed → closed, otherwise → open)
+// - If no route param, default to open on desktop, closed on mobile
+const hasExplicitRouteParam = query.has('route');
+const shouldSidebarBeOpen = hasExplicitRouteParam
+	? routeState.sidebarOpen
+	: !isMobile;
+
 const initialState: UIState = {
 	/**
 	 * Don't show certain modals after a page refresh.
@@ -165,9 +168,8 @@ const initialState: UIState = {
 		!isSeamlessMode &&
 		// We do not expect to render the Playground app UI in an iframe.
 		!isEmbeddedInAnIframe &&
-		// If there's an explicit route param requesting sidebar open, respect it.
-		// Otherwise default to closed on all viewport sizes.
-		hasExplicitRouteParam,
+		// Use explicit route param if provided, otherwise default based on viewport.
+		shouldSidebarBeOpen,
 	siteManagerSection: routeState.section,
 };
 

plugins/blocky-formats

@@ -0,0 +1 @@
+Subproject commit 372b8820fff1d49f24f865a6178d059a53e5df5b

pma/mysql-sqlite-network-proxy

@@ -0,0 +1 @@
+Subproject commit d0f8a46bfa5d4465e1bf87f50d401d71ff7b6d2d