From e1be4587880e67a100fa89fd8a6e5ef51efb2d0e Mon Sep 17 00:00:00 2001 From: Jiang Bohan Date: Wed, 6 May 2026 11:53:08 +0800 Subject: [PATCH] fix(projects): show URL tooltip on already-attached repos in Add Resource list MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The repo button in the Add Resource popover used the native `disabled` attribute when a repo was already attached. Browsers suppress pointer events on disabled form controls, so the tooltip on the URL text never fired for attached rows — the issue spec calls out "hovering over any URL should also show the complete URL in a tooltip". Switch to `aria-disabled` plus a click guard so the row still announces as disabled to assistive tech, looks the same visually, and is no longer click-able, but hover still reaches the tooltip trigger. Co-authored-by: multica-agent --- .../projects/components/project-resources-section.tsx | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/packages/views/projects/components/project-resources-section.tsx b/packages/views/projects/components/project-resources-section.tsx index 3d895976c..58c100320 100644 --- a/packages/views/projects/components/project-resources-section.tsx +++ b/packages/views/projects/components/project-resources-section.tsx @@ -118,16 +118,21 @@ export function ProjectResourcesSection({ projectId }: { projectId: string }) {
{workspace.repos.map((repo) => { const isAttached = attachedUrls.has(repo.url); + const isDisabled = isAttached || createResource.isPending; return ( + // Use aria-disabled instead of the native `disabled` attribute so + // hover events still reach the tooltip trigger on attached rows + // (browsers suppress pointer events on disabled form controls).