cmux NEXT: minimap rail and Apps chips keep the shared scroller (fixes the check-scrollbars base red)

Merged
TE
15d ago
+0-5

cmux-next checks fails on feat-cmux-next at check-scrollbars: 13 hits over its ceiling of 10. Three are web rules hiding a scrollbar, which SCROLLBARS-THIN forbids (scrollbar-width and ::-webkit-scrollbar turn WebKit's overlay scroller into a custom one):

  • threadMinimap.css: the minimap list (#22879) set scrollbar-width: none and ::-webkit-scrollbar { display: none }.
  • pages/apps/styles.css: the narrow Discover chips strip set scrollbar-width: none.

Both now keep the shared thin overlay scroller, hidden until the user scrolls. The minimap keeps its edge fades and overscroll-behavior: contain. Removing only the minimap's two would leave 11, still over the ceiling, so the chips line goes too. The check is now at 10 (warnings only; the rest are Swift ratchet debt).

Tests

  • SCROLLBARS_WARN_MAX=10 scripts/cmux-next/check-scrollbars.sh on this branch: passes (10 warnings), and on feat-cmux-next it fails (13 hits).
  • No test or gallery asserts these scrollbars are hidden.

No capture: webview CSS only; a scroller that shows only while scrolling, no change at rest.

Agent session: cosmic-gray

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.


Summary by cubic

Fixes the cmux-next check-scrollbars failure (13 hits over its ceiling of 10) by stopping the minimap rail and Apps Discover chips from hiding their scrollbars.

Both hid them with scrollbar-width: none and, for the minimap, ::-webkit-scrollbar { display: none } — rules that turn WebKit's overlay scroller into a custom one, which SCROLLBARS-THIN forbids. They now keep the shared thin overlay scroller, hidden until the user scrolls, so there's no visual change at rest. The minimap keeps its edge fades and overscroll-behavior: contain; the check drops to 10 warnings.

Tests

  • SCROLLBARS_WARN_MAX=10 scripts/cmux-next/check-scrollbars.sh passes on this branch and fails on feat-cmux-next (13 hits).
  • No test or gallery asserts these scrollbars are hidden.