From c4dd45f9ace334bf330c0f34ed95e06d5cc211a9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20Flor=C3=AAncio?= <carlosmflorencio@gmail.com>
Date: Mon, 13 Jul 2026 13:56:55 +0100
Subject: [PATCH] fix: use long press for mobile task actions (#1673)

---
 apps/web/components/task/task-item.tsx        | 10 +++-
 .../task/mobile-external-link-menu.spec.ts    | 16 ++++-
 .../task/mobile-sidebar-task-actions.spec.ts  | 60 +++++++++++++++++++
 3 files changed, 82 insertions(+), 4 deletions(-)
 create mode 100644 apps/web/e2e/tests/task/mobile-sidebar-task-actions.spec.ts

diff --git a/apps/web/components/task/task-item.tsx b/apps/web/components/task/task-item.tsx
index 894e380ec2..19b53d3131 100644
--- a/apps/web/components/task/task-item.tsx
+++ b/apps/web/components/task/task-item.tsx
@@ -273,9 +273,12 @@ function TaskItemStatsRow({
 function DiffStatsRight({ diffStats, menuOpen }: { diffStats: DiffStats; menuOpen: boolean }) {
   return (
     <div
+      data-testid="sidebar-task-diff-stats"
       className={cn(
         "shrink-0 self-center font-mono text-[11px] transition-opacity duration-100",
-        menuOpen ? "opacity-0" : "group-hover:opacity-0 group-focus-within:opacity-0",
+        menuOpen
+          ? "opacity-0"
+          : "[@media(hover:hover)]:group-hover:opacity-0 group-focus-within:opacity-0",
       )}
     >
       <span className="text-emerald-500">+{diffStats.additions}</span>{" "}
@@ -558,15 +561,16 @@ function TaskMenuButton({ visible }: { visible: boolean }) {
     <div
       className={cn(
         "self-center shrink-0 flex items-center transition-opacity duration-100",
+        !visible && "[@media(hover:none)]:hidden",
         visible
           ? "opacity-100"
-          : "opacity-0 pointer-events-none group-hover:opacity-100 group-hover:pointer-events-auto group-focus-within:opacity-100 group-focus-within:pointer-events-auto [@media(hover:none)]:opacity-100 [@media(hover:none)]:pointer-events-auto",
+          : "opacity-0 pointer-events-none [@media(hover:hover)]:group-hover:opacity-100 [@media(hover:hover)]:group-hover:pointer-events-auto group-focus-within:opacity-100 group-focus-within:pointer-events-auto",
       )}
     >
       <button
         type="button"
         className={cn(
-          "flex h-6 w-6 items-center justify-center rounded-md cursor-pointer",
+          "flex size-6 items-center justify-center rounded-md cursor-pointer",
           "text-muted-foreground hover:text-foreground hover:bg-foreground/10",
           "focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring transition-colors",
         )}
diff --git a/apps/web/e2e/tests/task/mobile-external-link-menu.spec.ts b/apps/web/e2e/tests/task/mobile-external-link-menu.spec.ts
index 8de47cc0ef..2e830a9b56 100644
--- a/apps/web/e2e/tests/task/mobile-external-link-menu.spec.ts
+++ b/apps/web/e2e/tests/task/mobile-external-link-menu.spec.ts
@@ -40,7 +40,21 @@ test.describe("Mobile sidebar — external link menu", () => {
       hasText: "Mobile external link task",
     });
     await expect(taskRow).toBeVisible({ timeout: 10_000 });
-    await taskRow.getByRole("button", { name: "Task actions" }).click();
+    await taskRow.dispatchEvent("pointerdown", {
+      pointerType: "touch",
+      isPrimary: true,
+      button: 0,
+      clientX: 120,
+      clientY: 240,
+    });
+    await testPage.waitForTimeout(1000);
+    await taskRow.dispatchEvent("pointerup", {
+      pointerType: "touch",
+      isPrimary: true,
+      button: 0,
+      clientX: 120,
+      clientY: 240,
+    });
 
     const linkTrigger = testPage.getByRole("menuitem", { name: /^Link$/ });
     await expect(linkTrigger).toBeVisible();
diff --git a/apps/web/e2e/tests/task/mobile-sidebar-task-actions.spec.ts b/apps/web/e2e/tests/task/mobile-sidebar-task-actions.spec.ts
new file mode 100644
index 0000000000..6cee7e72e9
--- /dev/null
+++ b/apps/web/e2e/tests/task/mobile-sidebar-task-actions.spec.ts
@@ -0,0 +1,60 @@
+import { test, expect } from "../../fixtures/test-base";
+import { SessionPage } from "../../pages/session-page";
+
+test.describe("Mobile sidebar task actions", () => {
+  test("opens task actions on long press without covering diff stats", async ({
+    testPage,
+    apiClient,
+    seedData,
+  }) => {
+    const taskTitle = "Mobile task with diff stats";
+    const task = await apiClient.createTaskWithAgent(
+      seedData.workspaceId,
+      taskTitle,
+      seedData.agentProfileId,
+      {
+        description: "/e2e:diff-update-setup",
+        workflow_id: seedData.workflowId,
+        workflow_step_id: seedData.startStepId,
+        repository_ids: [seedData.repositoryId],
+        executor_profile_id: seedData.worktreeExecutorProfileId,
+      },
+    );
+
+    await testPage.goto(`/t/${task.id}`);
+    const session = new SessionPage(testPage);
+    await session.waitForLoad();
+    await expect(
+      session.chat.getByText("diff-update-setup complete", { exact: false }),
+    ).toBeVisible({
+      timeout: 60_000,
+    });
+
+    await testPage.getByTestId("mobile-session-menu").click();
+    const sheet = testPage.getByRole("dialog");
+    const taskRow = sheet.getByTestId("sidebar-task-item").filter({ hasText: taskTitle });
+    const diffStats = taskRow.getByTestId("sidebar-task-diff-stats");
+    const actions = taskRow.getByRole("button", { name: "Task actions" });
+
+    await expect(diffStats).toBeVisible({ timeout: 15_000 });
+    await expect(actions).toBeHidden();
+
+    await taskRow.dispatchEvent("pointerdown", {
+      pointerType: "touch",
+      isPrimary: true,
+      button: 0,
+      clientX: 120,
+      clientY: 240,
+    });
+    await testPage.waitForTimeout(1000);
+    await taskRow.dispatchEvent("pointerup", {
+      pointerType: "touch",
+      isPrimary: true,
+      button: 0,
+      clientX: 120,
+      clientY: 240,
+    });
+
+    await expect(testPage.getByRole("menuitem", { name: "Archive" })).toBeVisible();
+  });
+});