Skip to content

fix: keep desktop sidebar independently scrollable - #150

Merged
rodriguecyber merged 1 commit into
Open-Source-Kigali:developfrom
bonheur15:fix/121-independent-sidebar-scroll
Aug 25, 2026
Merged

fix: keep desktop sidebar independently scrollable#150
rodriguecyber merged 1 commit into
Open-Source-Kigali:developfrom
bonheur15:fix/121-independent-sidebar-scroll

Conversation

@bonheur15

@bonheur15 bonheur15 commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • keep the desktop sidebar pinned to the viewport while the page scrolls
  • constrain the navigation region so long navigation scrolls independently
  • keep the sidebar header and footer fixed and reachable
  • preserve the fixed mobile drawer behavior

Type of change

  • Bug fix

Areas touched

  • apps/web (React / Vite)

Related issues

Closes #121

How to test

  1. Open a long dashboard page at a desktop breakpoint and scroll the page; the sidebar should remain in place.
  2. Reduce the viewport height until the navigation overflows; only the navigation region should scroll and the footer should remain visible.
  3. Switch to a mobile breakpoint, open and close the navigation drawer, and confirm its navigation remains scrollable.

Checklist

  • Change is focused and stays within the existing modular monolith boundaries.
  • Ran the relevant checks locally.
  • No documentation, protocol, or module-boundary changes are required.

Local checks

  • npm run lint --workspace=@docksight/web (passes with three existing warnings in unrelated files)
  • npm run test --workspace=@docksight/web (44 tests passed)
  • npm run build --workspace=@docksight/web
  • git diff --check

Screenshots

Desktop — constrained viewport

The navigation region has its own scrollbar while the account, theme, and version footer remains visible.

Desktop sidebar with independently scrollable navigation

Mobile drawer

The existing mobile drawer remains full-height and keeps its footer controls reachable.

Mobile navigation drawer

@rodriguecyber
rodriguecyber merged commit 8619107 into Open-Source-Kigali:develop Aug 25, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Make the desktop sidebar scroll independently

2 participants