From fd46e4f0b5ccfb805278fa8221acd5258aef2264 Mon Sep 17 00:00:00 2001
From: Chris Parsons <438676+chrisparsons83@users.noreply.github.com>
Date: Mon, 23 Feb 2026 22:25:28 -0800
Subject: [PATCH] Show all draft picks in sidebar with scroll instead of last
 10 (#33)

- Remove 10-pick limit in SidebarRecentPicks
- Fix scroll by using max-h-[45vh] on AccordionContent instead of the
  broken flex-1 approach (AccordionPrimitive.Content has overflow-hidden
  hardcoded, making flex-1 and overflow-y-auto ineffective without a
  bounded height)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 app/components/DraftSidebar.tsx                  | 8 ++++----
 app/components/draft/SidebarRecentPicks.tsx      | 1 -
 app/routes/leagues/$leagueId.draft.$seasonId.tsx | 2 +-
 3 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/app/components/DraftSidebar.tsx b/app/components/DraftSidebar.tsx
index 7e346e8..dd6f9ae 100644
--- a/app/components/DraftSidebar.tsx
+++ b/app/components/DraftSidebar.tsx
@@ -86,11 +86,11 @@ export function DraftSidebar({
           </AccordionItem>
 
           {/* Recent Picks Section */}
-          <AccordionItem value="recent-picks" className="border-0 flex-1 overflow-hidden flex flex-col">
-            <AccordionTrigger className="px-4 py-3 hover:no-underline bg-muted/50 hover:bg-muted flex-shrink-0">
-              <h2 className="font-semibold text-sm">Recent Picks</h2>
+          <AccordionItem value="recent-picks" className="border-0">
+            <AccordionTrigger className="px-4 py-3 hover:no-underline bg-muted/50 hover:bg-muted">
+              <h2 className="font-semibold text-sm">Picks</h2>
             </AccordionTrigger>
-            <AccordionContent className="pb-0 flex-1 overflow-y-auto">
+            <AccordionContent className="pb-0 overflow-y-auto max-h-[45vh]">
               {recentPicksSection}
             </AccordionContent>
           </AccordionItem>
diff --git a/app/components/draft/SidebarRecentPicks.tsx b/app/components/draft/SidebarRecentPicks.tsx
index b8b91ad..fa4cf04 100644
--- a/app/components/draft/SidebarRecentPicks.tsx
+++ b/app/components/draft/SidebarRecentPicks.tsx
@@ -29,7 +29,6 @@ export function SidebarRecentPicks({ picks }: SidebarRecentPicksProps) {
           {picks
             .slice()
             .reverse()
-            .slice(0, 10)
             .map((pick) => (
               <div
                 key={pick.id}
diff --git a/app/routes/leagues/$leagueId.draft.$seasonId.tsx b/app/routes/leagues/$leagueId.draft.$seasonId.tsx
index 7cd6e81..482124a 100644
--- a/app/routes/leagues/$leagueId.draft.$seasonId.tsx
+++ b/app/routes/leagues/$leagueId.draft.$seasonId.tsx
@@ -1263,7 +1263,7 @@ export default function DraftRoom() {
                 )}
 
                 <section>
-                  <h2 className="font-semibold text-sm mb-2">Recent Picks</h2>
+                  <h2 className="font-semibold text-sm mb-2">Picks</h2>
                   <SidebarRecentPicks picks={picks} />
                 </section>