fix(ui): uses overlay link pattern for ItemRow

- replaces stretched pseudo-element with separate overlay <a>

- fixes truncate clipping ::after (overflow:hidden conflict)

- conditionally renders link only for valid GitHub URLs

- adds aria-label with repo, number, and title context

Author: wgordon17

src/app/components/dashboard/ItemRow.tsx

@@ -28,6 +28,19 @@ export default function ItemRow(props: ItemRowProps) {
         transition-colors
         ${isCompact() ? "px-4 py-2" : "px-4 py-3"}`}
     >
+      {/* Overlay link — covers entire row; interactive children use relative z-10 */}
+      <Show when={safeUrl()}>
+        {(url) => (
+          <a
+            href={url()}
+            target="_blank"
+            rel="noopener noreferrer"
+            class="absolute inset-0"
+            aria-label={`${props.repo} #${props.number}: ${props.title}`}
+          />
+        )}
+      </Show>
+
       {/* Repo badge */}
       <Show when={!props.hideRepo}>
         <span
@@ -46,14 +59,9 @@ export default function ItemRow(props: ItemRowProps) {
           <span class="text-base-content/60 shrink-0">
             #{props.number}
           </span>
-          <a
-            href={safeUrl()}
-            target="_blank"
-            rel="noopener noreferrer"
-            class="font-medium text-base-content truncate after:absolute after:inset-0"
-          >
+          <span class="font-medium text-base-content truncate">
             {props.title}
-          </a>
+          </span>
         </div>
 
         {/* Labels row */}
@@ -77,9 +85,9 @@ export default function ItemRow(props: ItemRowProps) {
           </div>
         </Show>
 
-        {/* Additional children slot */}
+        {/* Additional children slot — z-10 to sit above stretched link */}
         <Show when={props.children !== undefined}>
-          <div class={isCompact() ? "mt-0.5" : "mt-1"}>{props.children}</div>
+          <div class={`relative z-10 ${isCompact() ? "mt-0.5" : "mt-1"}`}>{props.children}</div>
         </Show>
       </div>
 

tests/components/ItemRow.test.tsx

@@ -54,23 +54,33 @@ describe("ItemRow", () => {
     screen.getByTestId("child-slot");
   });
 
+  it("children slot sits above overlay link (relative z-10)", () => {
+    const { container } = render(() => (
+      <ItemRow {...defaultProps}>
+        <span data-testid="child-slot">extra content</span>
+      </ItemRow>
+    ));
+    const childWrapper = container.querySelector("[data-testid='child-slot']")!.parentElement!;
+    expect(childWrapper.className).toContain("relative");
+    expect(childWrapper.className).toContain("z-10");
+  });
+
   it("does not render children slot when not provided", () => {
     render(() => <ItemRow {...defaultProps} />);
     expect(screen.queryByTestId("child-slot")).toBeNull();
   });
 
-  it("title is a real link with correct href and target", () => {
+  it("renders overlay link with correct href, target, rel, and aria-label", () => {
     render(() => <ItemRow {...defaultProps} />);
-    const link = screen.getByText("Fix a bug").closest("a")!;
+    const link = screen.getByRole("link", { name: /octocat\/Hello-World #42: Fix a bug/ });
     expect(link.getAttribute("href")).toBe(defaultProps.url);
     expect(link.getAttribute("target")).toBe("_blank");
     expect(link.getAttribute("rel")).toBe("noopener noreferrer");
   });
 
-  it("title link has no href for non-GitHub URLs", () => {
+  it("does not render overlay link for non-GitHub URLs", () => {
     render(() => <ItemRow {...defaultProps} url="https://evil.com/bad" />);
-    const link = screen.getByText("Fix a bug").closest("a")!;
-    expect(link.getAttribute("href")).toBeNull();
+    expect(screen.queryByRole("link")).toBeNull();
   });
 
   it("calls onIgnore when ignore button is clicked", async () => {
@@ -84,15 +94,11 @@ describe("ItemRow", () => {
     expect(onIgnore).toHaveBeenCalledOnce();
   });
 
-  it("ignore button does not navigate (sits above stretched link)", async () => {
-    const user = userEvent.setup();
-    const onIgnore = vi.fn();
-    render(() => <ItemRow {...defaultProps} onIgnore={onIgnore} />);
-
+  it("ignore button has relative z-10 to sit above overlay link", () => {
+    render(() => <ItemRow {...defaultProps} />);
     const ignoreBtn = screen.getByLabelText(/Ignore #42/i);
-    await user.click(ignoreBtn);
-
-    expect(onIgnore).toHaveBeenCalledOnce();
+    expect(ignoreBtn.className).toContain("relative");
+    expect(ignoreBtn.className).toContain("z-10");
   });
 
   it("applies compact padding in compact density", () => {