From 8fe334b056d4ab83e51a2abf2ca4d1ac8982770a Mon Sep 17 00:00:00 2001 From: Alli Date: Mon, 15 Jun 2026 02:23:38 +0000 Subject: [PATCH] fix(desktop): inset hover-reveal trigger past the adjacent scrollbar (#44159) The collapsed-pane hover-reveal trigger strip (14px wide, 6px edge gutter) overlapped the neighboring scroller's 8px .scrollbar-dt scrollbar, which sits flush with the window edge when the rail panes are collapsed. Hovering the scrollbar revealed the file browser over it, and clicks on the overlapped band hit the trigger instead of the scrollbar thumb. Widen the edge gutter to calc(0.5rem + 2px) so the strip clears the scrollbar (rem-coupled to the .scrollbar-dt width) while still covering the OS window-resize grab area inset. Part of #44140 (item 2). Co-authored-by: AIalliAI <285906080+AIalliAI@users.noreply.github.com> Co-authored-by: Claude Fable 5 --- apps/desktop/src/components/pane-shell/pane-shell.tsx | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/apps/desktop/src/components/pane-shell/pane-shell.tsx b/apps/desktop/src/components/pane-shell/pane-shell.tsx index 61e7e6969ad..eaa4bf21363 100644 --- a/apps/desktop/src/components/pane-shell/pane-shell.tsx +++ b/apps/desktop/src/components/pane-shell/pane-shell.tsx @@ -80,9 +80,12 @@ const HOVER_REVEAL_EASE = 'cubic-bezier(0.32,0.72,0,1)' // Offset shadow lifting the revealed panel off the content (same both sides; // the mirror axis is offset-x, which is 0). Same color on light + dark. const HOVER_REVEAL_SHADOW = '0px -18px 18px -5px #00000012' -// Edge trigger strip, inset past the OS window-resize grab area. +// Edge trigger strip, inset past the OS window-resize grab area AND the +// adjacent pane's scrollbar (0.5rem, .scrollbar-dt) — the strip overlays the +// neighboring scroller's edge, so any overlap makes the scrollbar reveal the +// pane on hover and swallow its clicks (#44140). const HOVER_REVEAL_TRIGGER_WIDTH = 14 -const HOVER_REVEAL_EDGE_GUTTER = 6 +const HOVER_REVEAL_EDGE_GUTTER = 'calc(0.5rem + 2px)' // Fired (window CustomEvent<{ id }>) to toggle a force-collapsed pane's reveal // from the keyboard, since its store-open toggle is a no-op while collapsed.