From 5e907783fd7d5f092d1eb8379a6efdfa55008d5d Mon Sep 17 00:00:00 2001
From: hjkwon <80230648+GagaKor@users.noreply.github.com>
Date: Tue, 27 Oct 2026 04:23:23 +0900
Subject: [PATCH] fix(linear): make new-project dialog popovers
 wheel-scrollable (#16413)

The Team, Lead, Members and Labels popovers in the Linear new-project
dialog are portaled outside the dialog, so its scroll-lock cancelled
native wheel input over their inner max-h-64 lists. Opt their
PopoverContent into the shared wheel shim via wheelScroll, and move the
p-1 padding onto the children so the layout is unchanged.

Fixes #16412

Co-authored-by: Neil <neil@stably.ai>
---
 .../task-page/linear/ProjectDialog.tsx        |  6 ++---
 .../task-page/linear/ProjectFields.tsx        | 24 +++++++++----------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/renderer/src/components/task-page/linear/ProjectDialog.tsx b/src/renderer/src/components/task-page/linear/ProjectDialog.tsx
index 38571cb7990..1ceae59890b 100644
--- a/src/renderer/src/components/task-page/linear/ProjectDialog.tsx
+++ b/src/renderer/src/components/task-page/linear/ProjectDialog.tsx
@@ -78,11 +78,11 @@ export function TaskPageLinearProjectDialog({
                     <ChevronDown className="size-3 flex-none text-muted-foreground" />
                   </Button>
                 </PopoverTrigger>
-                <PopoverContent align="start" className="w-72 p-1">
-                  <div className="px-2 py-1.5 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
+                <PopoverContent align="start" className="w-72" wheelScroll>
+                  <div className="mx-1 mt-1 px-2 py-1.5 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
                     {translate('auto.components.TaskPage.a98cbe7664', 'Team')}
                   </div>
-                  <div className="max-h-64 overflow-y-auto scrollbar-sleek">
+                  <div className="mx-1 mb-1 max-h-64 overflow-y-auto scrollbar-sleek">
                     {availableTeams.map((team) => (
                       <button
                         key={team.id}
diff --git a/src/renderer/src/components/task-page/linear/ProjectFields.tsx b/src/renderer/src/components/task-page/linear/ProjectFields.tsx
index db743708706..aac14102973 100644
--- a/src/renderer/src/components/task-page/linear/ProjectFields.tsx
+++ b/src/renderer/src/components/task-page/linear/ProjectFields.tsx
@@ -117,16 +117,16 @@ export function TaskPageLinearProjectFields({
               <ChevronDown className="size-3 text-muted-foreground/70" />
             </button>
           </PopoverTrigger>
-          <PopoverContent align="start" className="w-64 p-1">
-            <div className="px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
+          <PopoverContent align="start" className="w-64" wheelScroll>
+            <div className="mx-1 mt-1 px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
               {translate('auto.components.TaskPage.34da8ac06c', 'Lead')}
             </div>
             {newLinearProjectMembers.loading ? (
-              <div className="flex items-center justify-center p-4">
+              <div className="mx-1 mb-1 flex items-center justify-center p-4">
                 <LoaderCircle className="size-4 animate-spin text-muted-foreground" />
               </div>
             ) : (
-              <div className="max-h-64 overflow-y-auto scrollbar-sleek">
+              <div className="mx-1 mb-1 max-h-64 overflow-y-auto scrollbar-sleek">
                 <button
                   type="button"
                   onClick={() => setNewLinearProjectLeadId(null)}
@@ -200,16 +200,16 @@ export function TaskPageLinearProjectFields({
               <ChevronDown className="size-3 text-muted-foreground/70" />
             </button>
           </PopoverTrigger>
-          <PopoverContent align="start" className="w-64 p-1">
-            <div className="px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
+          <PopoverContent align="start" className="w-64" wheelScroll>
+            <div className="mx-1 mt-1 px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
               {translate('auto.components.TaskPage.d6cda23ef1', 'Members')}
             </div>
             {newLinearProjectMembers.loading ? (
-              <div className="flex items-center justify-center p-4">
+              <div className="mx-1 mb-1 flex items-center justify-center p-4">
                 <LoaderCircle className="size-4 animate-spin text-muted-foreground" />
               </div>
             ) : (
-              <div className="max-h-64 overflow-y-auto scrollbar-sleek">
+              <div className="mx-1 mb-1 max-h-64 overflow-y-auto scrollbar-sleek">
                 {newLinearProjectMembers.data.map((member) => {
                   const selected = newLinearProjectMemberIds.includes(member.id)
                   return (
@@ -268,16 +268,16 @@ export function TaskPageLinearProjectFields({
               <ChevronDown className="size-3 text-muted-foreground/70" />
             </button>
           </PopoverTrigger>
-          <PopoverContent align="start" className="w-64 p-1">
-            <div className="px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
+          <PopoverContent align="start" className="w-64" wheelScroll>
+            <div className="mx-1 mt-1 px-2 py-1 text-[10px] font-semibold uppercase tracking-wider text-muted-foreground">
               {translate('auto.components.TaskPage.d0ca4aa1d0', 'Labels')}
             </div>
             {newLinearProjectLabels.loading ? (
-              <div className="flex items-center justify-center p-4">
+              <div className="mx-1 mb-1 flex items-center justify-center p-4">
                 <LoaderCircle className="size-4 animate-spin text-muted-foreground" />
               </div>
             ) : (
-              <div className="max-h-64 overflow-y-auto scrollbar-sleek">
+              <div className="mx-1 mb-1 max-h-64 overflow-y-auto scrollbar-sleek">
                 {newLinearProjectLabels.data.length === 0 ? (
                   <div className="px-2 py-2 text-xs text-muted-foreground">
                     {translate('auto.components.TaskPage.af9e877f30', 'No labels')}