Skip to content

fix(web): restore independent reader pane scrollbars - #12

Merged
sommio merged 3 commits into
developfrom
fix/reader-pane-scrollbar
Apr 19, 2026
Merged

fix(web): restore independent reader pane scrollbars#12
sommio merged 3 commits into
developfrom
fix/reader-pane-scrollbar

Conversation

@sommio

@sommio sommio commented Apr 19, 2026

Copy link
Copy Markdown
Owner

Summary

Restores the desktop reader's two-pane scrolling contract so the page no
longer acts as the primary scroll container and both panes keep stable,
discoverable scroll roots.

What changed

  • tighten the shared �0ScrollArea�0 primitive into a flex-owned scroll root
    with mounted scrollbar structure and stable test hooks
  • move the detail pane onto the shared scroll contract, keep headers
    outside the scroll body, and preserve centered placeholder states
  • add SSR and Playwright coverage for pane-owned scrolling, fixed headers,
    and pending or unavailable detail states
  • record the implementation plan and the paired knowledge doc for future
    reader work

Test plan

  • pnpm --filter web test -- --runInBand app/page.spec.tsx
  • pnpm --filter web exec playwright test e2e/home.spec.ts
  • pnpm --filter @repo/ui typecheck
  • pnpm --filter web typecheck
  • pnpm --filter @repo/ui lint
  • pnpm --filter web lint

Compound Engineering
Codex

sommio added 3 commits April 19, 2026 16:52
Rebuild the reader panes around a shared ScrollArea contract so the list
and detail columns own scrolling inside the fixed shell again. Add SSR and
Playwright regression coverage for pane-local overflow, fixed headers, and
state preservation, then tune the custom scrollbar styling to stay subtle
until hover or active scrolling.
Rebuild the full-height flex chain for placeholder detail states so
empty and unavailable views stay vertically centered inside the shared
scroll area.

Add SSR assertions to guard the placeholder layout contract against
future scrollbar refactors.
@sommio
sommio merged commit eff82af into develop Apr 19, 2026
5 checks passed
@sommio
sommio deleted the fix/reader-pane-scrollbar branch April 19, 2026 13:01
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.

1 participant