fix(layout): sidebar scrollbar affordance on short viewports (PMC-17) #27

Closed
David wants to merge 0 commits from fix/pmc-17-dashboard-scrollbar into main
Owner

Summary

Closes PMC-17 (Dashboard UI). Of the five sub-tasks, PMC-18/PMC-19/PMC-21 were already resolved in main and closed in the audit pass; PMC-20 was closed as blocked-on-server (no /api/v1/reports/dashboard endpoint exists yet). Only PMC-22 needed a code change.

PMC-22: swap the desktop sidebar's .scrollbar-hide for .scrollbar-thin so a long nav list surfaces a slim scrollbar when the viewport is short instead of silently clipping. At full height nothing visually changes.

Test plan

  • cargo check --features web clean
  • In a short viewport (e.g. devtools 600px height) the sidebar shows a thin scrollbar; long-page main scroll still works independently
  • At normal height the sidebar shows no scrollbar (parity with previous behaviour)

🤖 Generated with Claude Code

## Summary Closes PMC-17 (Dashboard UI). Of the five sub-tasks, PMC-18/PMC-19/PMC-21 were already resolved in main and closed in the audit pass; PMC-20 was closed as blocked-on-server (no `/api/v1/reports/dashboard` endpoint exists yet). Only PMC-22 needed a code change. PMC-22: swap the desktop sidebar's `.scrollbar-hide` for `.scrollbar-thin` so a long nav list surfaces a slim scrollbar when the viewport is short instead of silently clipping. At full height nothing visually changes. ## Test plan - [ ] `cargo check --features web` clean - [ ] In a short viewport (e.g. devtools 600px height) the sidebar shows a thin scrollbar; long-page main scroll still works independently - [ ] At normal height the sidebar shows no scrollbar (parity with previous behaviour) 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(layout): show scrollbar affordance on short-viewport sidebars (PMC-22)
Some checks failed
Check / clippy + fmt + tests (pull_request) Failing after 4s
Create release / Create release from merged PR (pull_request) Has been skipped
404783ef4f
The desktop sidebar previously used \`.scrollbar-hide\` so a long nav list could scroll without visual chrome. At narrow viewport heights the nav overflows but the hidden bar leaves the user no signal that there's more below, and the main panel's own scrollbar makes it look like the bottom nav items are simply missing.

Switching the sidebar to \`.scrollbar-thin\` keeps the column visually quiet at full height (where the nav fits and nothing scrolls) but surfaces a slim affordance the moment the viewport gets short enough to push items below the fold.

#PMC-22 State Done
vas2000-work closed this pull request 2026-05-21 01:21:54 +02:00
Some checks failed
Check / clippy + fmt + tests (pull_request) Failing after 4s
Required
Details
Create release / Create release from merged PR (pull_request) Has been skipped

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-apps!27
No description provided.