From 2d87b3e9f77ea1b61aeffc552d64f2a7f282139d Mon Sep 17 00:00:00 2001
From: Shiva Gupta <shivascalerite@gmail.com>
Date: Fri, 12 Dec 2025 02:27:02 +0530
Subject: [PATCH] fix(tasks): use uuid instead of id for dialog open condition

Taskwarrior assigns id=0 to all completed and deleted tasks since they
are removed from the active task list. Using task.id for the dialog's
open condition caused multiple dialogs to open simultaneously when
clicking on any non-pending task, resulting in flickering and blackout.

Replaced _selectedTask?.id === task.id with
_selectedTask?.uuid === task.uuid since uuid is unique for every
task regardless of status.

Fixes: #271
---
 frontend/src/components/HomeComponents/Tasks/Tasks.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/frontend/src/components/HomeComponents/Tasks/Tasks.tsx b/frontend/src/components/HomeComponents/Tasks/Tasks.tsx
index d9bf20e5..5043fba0 100644
--- a/frontend/src/components/HomeComponents/Tasks/Tasks.tsx
+++ b/frontend/src/components/HomeComponents/Tasks/Tasks.tsx
@@ -998,7 +998,7 @@ export const Tasks = (
                             selectedIndex={selectedIndex}
                             task={task}
                             isOpen={
-                              _isDialogOpen && _selectedTask?.id === task.id
+                              _isDialogOpen && _selectedTask?.uuid === task.uuid
                             }
                             onOpenChange={handleDialogOpenChange}
                             editState={editState}