From 9be9925467f336efbc5139bd8d7b69616721dd69 Mon Sep 17 00:00:00 2001
From: Brooklyn Nicholson <brooklyn.bb.nicholson@gmail.com>
Date: Thu, 13 Aug 2026 02:09:50 -0500
Subject: [PATCH] fix(desktop): themed fade scrollbar on the virtual list, not
 overlay
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

scrollbar-overlay opts out of the themed thin scrollbar; on Windows there
are no native overlay scrollbars, so Chromium painted the classic
always-visible gutter instead — a permanent scrollbar next to the recents
list. The themed fade bar reserves its 4px on every platform but stays
invisible until hover, and the wrapper no longer stacks a second scroller,
which is what the overlay class was originally working around.
---
 .../src/app/chat/sidebar/virtual-session-list.tsx   | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/apps/desktop/src/app/chat/sidebar/virtual-session-list.tsx b/apps/desktop/src/app/chat/sidebar/virtual-session-list.tsx
index cb9f99714d552..f99edb5f020da 100644
--- a/apps/desktop/src/app/chat/sidebar/virtual-session-list.tsx
+++ b/apps/desktop/src/app/chat/sidebar/virtual-session-list.tsx
@@ -152,13 +152,14 @@ export const VirtualSessionList: FC<VirtualSessionListProps> = ({
   // just consume that context via useSortable.
   return (
     <div
-      // `scrollbar-overlay`, not `scrollbar-fade`: the themed (classic)
-      // scrollbar reserves a 4px gutter, which shaved this list's rows
-      // narrower than the pinned/non-virtual lists above it and left dead
-      // space along the right edge. Overlay keeps native macOS behavior —
-      // zero gutter, thumb draws over content only while scrolling.
+      // scrollbar-fade, NOT scrollbar-overlay: overlay opts out of the themed
+      // thin scrollbar entirely, and on Windows (no native overlay scrollbars)
+      // Chromium then paints the classic always-visible gutter. The themed
+      // fade bar reserves its 4px on every platform but stays invisible until
+      // hover — and the wrapper no longer stacks a second scroller, so the
+      // double-gutter this class change was reaching for is already gone.
       className={cn(
-        'scrollbar-overlay relative min-h-0 flex-1 overflow-x-hidden overflow-y-auto overscroll-contain',
+        'scrollbar-fade relative min-h-0 flex-1 overflow-x-hidden overflow-y-auto overscroll-contain',
         className
       )}
       ref={scrollerRef}