From fa439d7c7b34a0cb07afd6cbece1313f6af55ba4 Mon Sep 17 00:00:00 2001 From: claude-stand Date: Thu, 2 Jul 2026 18:19:59 +0300 Subject: [PATCH] fix(ui): match sidebar toggle breakpoint to navbar (md) so the tablet drawer opens MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Follow-up to the navbar sm->md change on this branch: the two header sidebar toggles were still gated at sm, so in the 768-991 band the DESKTOP toggle was shown while the navbar used the MOBILE drawer collapse state — clicking it flipped the wrong atom and the drawer could not be opened (sidebar unreachable at 768/820, caught by QA). Gate the mobile toggle hiddenFrom=md and the desktop toggle visibleFrom=md so the mobile toggle drives the drawer across the whole tablet band. Co-Authored-By: Claude Opus 4.8 (1M context) --- .../src/components/layouts/global/app-header.tsx | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/apps/client/src/components/layouts/global/app-header.tsx b/apps/client/src/components/layouts/global/app-header.tsx index 96b0a75d..bf1add95 100644 --- a/apps/client/src/components/layouts/global/app-header.tsx +++ b/apps/client/src/components/layouts/global/app-header.tsx @@ -53,7 +53,13 @@ export function AppHeader() { aria-label={t("Sidebar toggle")} opened={mobileOpened} onClick={toggleMobile} - hiddenFrom="sm" + // Must match the AppShell navbar breakpoint (md). The navbar + // collapses to the MOBILE drawer below md, so the mobile toggle + // (which flips mobileOpened) must be the one visible across the + // whole md navbar change). + hiddenFrom="md" size="sm" /> @@ -63,7 +69,7 @@ export function AppHeader() { aria-label={t("Sidebar toggle")} opened={desktopOpened} onClick={toggleDesktop} - visibleFrom="sm" + visibleFrom="md" size="sm" />