fix(web): sidebar session count + minimal auto-hiding scrollbars - #19
Merged
Merged
Conversation
The sidebar "Sessions" badge was wired to the live active tally (running + awaiting) sourced only from the channel map, so a user with only DONE sessions in the persisted registry saw 0. Move the session registry load into the (app) layout so the shell sees it, and badge the union of registry + live session IDs — the exact set the dashboard lists, so the count matches the page. The dashboard page now reads data.sessions from merged layout data (one fetch); the system bar keeps its live active-agent tally. Also replace the default heavy/always-on scrollbars with an app-wide minimal, auto-hiding treatment: the thumb is transparent at rest and resolves only while the pointer is over (or focus is within) the scroll region, ~4px inside a 10px track. Driven by new --scrollbar-thumb tokens that re-theme under the light theme; covers WebKit and Firefox.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small UI fixes from the command-center dashboard.
1. "Sessions" badge showed 0 with sessions present
The sidebar badge was wired to the live active tally (
running + awaiting) sourced only from the channel map, but DONE sessions live in the persisted registry (data.sessions) — which only the dashboard page loaded, not the shell. So the sidebar always saw 0.listSessionsinto the(app)layout load so the shell sees the registry (one fetch; the dashboard page now readsdata.sessionsfrom merged layout data).2. Inner scrollbar too heavy / always visible
App-wide minimal, auto-hiding scrollbars: the thumb is transparent at rest and resolves only while the pointer is over (or focus is within) the scroll region (~4px visible inside a 10px track). Driven by new
--scrollbar-thumb/--scrollbar-thumb-hovertokens that re-theme under the light theme; covers WebKit/Chromium and Firefox.Gates
pnpm --filter web check— 0 errorspnpm --filter web lint— cleanpnpm --filter web test— 217 passingNote
The registry list now refreshes on full navigation rather than every client-side return to the dashboard, but the live WS overlay keeps statuses current and newly-launched sessions appear immediately via the live map.