fix(admin): keep responsive navigation left-aligned (CoreBunch#218)

DavidBabinec · web-flow · commit 960fdaa55e4f · 2026-07-13T20:37:39.000+02:00
diff --git a/src/admin/pages/site/toolbar/Toolbar.module.css b/src/admin/pages/site/toolbar/Toolbar.module.css
@@ -286,13 +286,6 @@
         padding: var(--space-4xs) var(--space-s);
     }
 
-    .siteName,
-    .siteNameSkeleton,
-    .siteFavicon,
-    .workspaceToolbarItems {
-        order: 1;
-    }
-
     .siteName {
         max-width: 104px;
         margin-right: var(--space-4xs);
@@ -320,7 +313,6 @@
 
     .adminLink,
     .activeSection {
-        order: 2;
         justify-content: center;
         width: 30px;
         min-width: 30px;
diff --git a/tests/e2e/admin-navigation.e2e.ts b/tests/e2e/admin-navigation.e2e.ts
@@ -113,6 +113,7 @@ test.describe('admin navigation', () => {
         page.getByTestId('account-menu-trigger'),
         'mobile toolbar account menu trigger',
       )
+      await expectNavigationBeforeToolbarTrailer(page)
     })
   })
 
@@ -508,6 +509,20 @@ async function expectLocatorContained(
   expect(box.x + box.width).toBeLessThanOrEqual(viewportWidth + 1)
 }
 
+async function expectNavigationBeforeToolbarTrailer(page: Page): Promise<void> {
+  const toolbar = page.getByTestId('toolbar')
+  const navigationBox = await toolbar.getByRole('link', { name: 'Site' }).boundingBox()
+  const settingsBox = await toolbar.getByTestId('toolbar-settings-btn').boundingBox()
+  if (!navigationBox || !settingsBox) {
+    throw new Error('mobile toolbar navigation or settings button had no bounding box')
+  }
+
+  const navigationPrecedesSettings =
+    navigationBox.y < settingsBox.y ||
+    (Math.abs(navigationBox.y - settingsBox.y) <= 1 && navigationBox.x < settingsBox.x)
+  expect(navigationPrecedesSettings).toBe(true)
+}
+
 async function expectStoredWorkspaceLayout(
   page: Page,
   expected: {
