fix(layout): main sidebar nav scrolls when it overflows #76

Merged

vas2000-work merged 1 commit from fix/main-sidebar-scroll into main

2026-06-06 00:54:06 +02:00

vas2000-work commented

2026-06-06 00:53:42 +02:00

Owner

The main app sidebar nav could overflow the viewport (many nav groups, or a short window), but the scroll lived only on the desktop aside. The mobile drawer aside has no overflow of its own and the shared SidebarContent did not put scroll on the nav, so the lower groups (Analytics / Admin) were unreachable.

Make the nav element the scroll container (flex-1 + min-h-0 + overflow-y-auto + overscroll-contain + scrollbar-thin) so it scrolls in both the desktop sidebar and the mobile drawer, with the version footer pinned below. Standard nested-flex scroll pattern.

Needs a gate run + a quick visual check at a short viewport / mobile width.

The main app sidebar nav could overflow the viewport (many nav groups, or a short window), but the scroll lived only on the desktop `aside`. The mobile drawer `aside` has no overflow of its own and the shared SidebarContent did not put scroll on the nav, so the lower groups (Analytics / Admin) were unreachable. Make the `nav` element the scroll container (flex-1 + min-h-0 + overflow-y-auto + overscroll-contain + scrollbar-thin) so it scrolls in both the desktop sidebar and the mobile drawer, with the version footer pinned below. Standard nested-flex scroll pattern. Needs a gate run + a quick visual check at a short viewport / mobile width.

vas2000-work added 1 commit

2026-06-06 00:53:42 +02:00

fix(layout): make the main sidebar nav scroll when it overflows

Create release / Create release from merged PR (pull_request) Has been skipped

Details

Check / clippy + fmt + tests (pull_request) Successful in 45s

Details

506e2d3708

The nav list could exceed the viewport (many groups, or a short window), but the scroll was only on the desktop aside; the mobile drawer aside has no overflow of its own, so its lower groups (Analytics / Admin) were unreachable. Make the nav element the scroll container (flex-1 + min-h-0 + overflow-y-auto + overscroll-contain) so it scrolls in both the desktop sidebar and the mobile drawer, with the version footer pinned below.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>