From 36338f3546f9e3db1306fc295a7e8d0365f7a73b Mon Sep 17 00:00:00 2001
From: AndyMik90 <andre@mikalsenutvikling.no>
Date: Tue, 16 Dec 2025 23:20:51 +0100
Subject: [PATCH] Refactor IdeationHeader and update handleDeleteSelected logic

- Simplified the IdeationHeader component by removing the Tooltip wrapper around the Delete button for a cleaner UI.
- Updated handleDeleteSelected to fetch the latest selected IDs from the store, ensuring it operates on the current state.
---
 .../components/ideation/IdeationHeader.tsx    | 23 ++++++++-----------
 .../components/ideation/hooks/useIdeation.ts  | 10 ++++----
 2 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/auto-claude-ui/src/renderer/components/ideation/IdeationHeader.tsx b/auto-claude-ui/src/renderer/components/ideation/IdeationHeader.tsx
index bf6ae373..5ec0a170 100644
--- a/auto-claude-ui/src/renderer/components/ideation/IdeationHeader.tsx
+++ b/auto-claude-ui/src/renderer/components/ideation/IdeationHeader.tsx
@@ -64,20 +64,15 @@ export function IdeationHeader({
               <Badge variant="secondary" className="mr-1">
                 {selectedCount} selected
               </Badge>
-              <Tooltip>
-                <TooltipTrigger asChild>
-                  <Button
-                    variant="outline"
-                    size="sm"
-                    className="text-destructive hover:bg-destructive hover:text-destructive-foreground"
-                    onClick={onDeleteSelected}
-                  >
-                    <Trash2 className="h-4 w-4 mr-1" />
-                    Delete
-                  </Button>
-                </TooltipTrigger>
-                <TooltipContent>Delete selected ideas</TooltipContent>
-              </Tooltip>
+              <Button
+                variant="outline"
+                size="sm"
+                className="text-destructive hover:bg-destructive hover:text-destructive-foreground"
+                onClick={onDeleteSelected}
+              >
+                <Trash2 className="h-4 w-4 mr-1" />
+                Delete
+              </Button>
               <Tooltip>
                 <TooltipTrigger asChild>
                   <Button
diff --git a/auto-claude-ui/src/renderer/components/ideation/hooks/useIdeation.ts b/auto-claude-ui/src/renderer/components/ideation/hooks/useIdeation.ts
index fda54ff0..425cb4aa 100644
--- a/auto-claude-ui/src/renderer/components/ideation/hooks/useIdeation.ts
+++ b/auto-claude-ui/src/renderer/components/ideation/hooks/useIdeation.ts
@@ -156,10 +156,12 @@ export function useIdeation(projectId: string, options: UseIdeationOptions = {})
     }
   };
 
-  const handleDeleteSelected = async () => {
-    if (selectedIds.size === 0) return;
-    await deleteMultipleIdeasForProject(projectId, Array.from(selectedIds));
-  };
+  const handleDeleteSelected = useCallback(async () => {
+    // Get fresh selectedIds from store to avoid stale closure
+    const currentSelectedIds = useIdeationStore.getState().selectedIds;
+    if (currentSelectedIds.size === 0) return;
+    await deleteMultipleIdeasForProject(projectId, Array.from(currentSelectedIds));
+  }, [projectId]);
 
   const handleSelectAll = useCallback((ideas: Idea[]) => {
     selectAllIdeas(ideas.map(idea => idea.id));