clean-up

Author: jonathanlab

src/renderer/components/TaskItem.tsx

@@ -47,7 +47,9 @@ function TaskItemComponent({
   const duplicateTask = useTaskStore((state) => state.duplicateTask);
   const setSelectedIndex = useTaskStore((state) => state.setSelectedIndex);
   const setHoveredIndex = useTaskStore((state) => state.setHoveredIndex);
-  const setContextMenuIndex = useTaskStore((state) => state.setContextMenuIndex);
+  const setContextMenuIndex = useTaskStore(
+    (state) => state.setContextMenuIndex,
+  );
   const selectedIndex = useTaskStore((state) => state.selectedIndex);
   const contextMenuIndex = useTaskStore((state) => state.contextMenuIndex);
   const createdAt = new Date(task.created_at);
@@ -223,16 +225,33 @@ function TaskItemComponent({
           <ContextMenu.Sub>
             <ContextMenu.SubTrigger>Move</ContextMenu.SubTrigger>
             <ContextMenu.SubContent>
-              <ContextMenu.Item onSelect={() => onMoveTask(index, 0)} shortcut="⌥ ⇧ ↑">
+              <ContextMenu.Item
+                onSelect={() => onMoveTask(index, 0)}
+                shortcut="⌥ ⇧ ↑"
+              >
                 Move to top
               </ContextMenu.Item>
-              <ContextMenu.Item onSelect={() => onMoveTask(index, Math.max(0, index - 1))} shortcut="⌥ ↑">
+              <ContextMenu.Item
+                onSelect={() => onMoveTask(index, Math.max(0, index - 1))}
+                shortcut="⌥ ↑"
+              >
                 Move up
               </ContextMenu.Item>
-              <ContextMenu.Item onSelect={() => onMoveTask(index, Math.min(filteredTasksLength - 1, index + 1))} shortcut="⌥ ↓">
+              <ContextMenu.Item
+                onSelect={() =>
+                  onMoveTask(
+                    index,
+                    Math.min(filteredTasksLength - 1, index + 1),
+                  )
+                }
+                shortcut="⌥ ↓"
+              >
                 Move down
               </ContextMenu.Item>
-              <ContextMenu.Item onSelect={() => onMoveTask(index, filteredTasksLength - 1)} shortcut="⌥ ⇧ ↓">
+              <ContextMenu.Item
+                onSelect={() => onMoveTask(index, filteredTasksLength - 1)}
+                shortcut="⌥ ⇧ ↓"
+              >
                 Move to bottom
               </ContextMenu.Item>
             </ContextMenu.SubContent>

src/renderer/components/TaskList.tsx

@@ -39,17 +39,14 @@ export function TaskList({ onSelectTask, onNewTask }: TaskListInternalProps) {
   // Move drag state to local state for better performance
   const [draggedTaskId, setDraggedTaskIdLocal] = useState<string | null>(null);
   const [dragOverIndex, setDragOverIndexLocal] = useState<number | null>(null);
-  const [dropPosition, setDropPositionLocal] = useState<"top" | "bottom" | null>(null);
+  const [dropPosition, setDropPositionLocal] = useState<
+    "top" | "bottom" | null
+  >(null);
 
   const fetchTasks = useTaskStore((state) => state.fetchTasks);
-  const deleteTask = useTaskStore((state) => state.deleteTask);
-  const duplicateTask = useTaskStore((state) => state.duplicateTask);
   const moveTask = useTaskStore((state) => state.moveTask);
   const setSelectedIndex = useTaskStore((state) => state.setSelectedIndex);
   const setHoveredIndex = useTaskStore((state) => state.setHoveredIndex);
-  const setContextMenuIndex = useTaskStore(
-    (state) => state.setContextMenuIndex,
-  );
   const setFilter = useTaskStore((state) => state.setFilter);
 
   const { setStatusBar, reset } = useStatusBarStore();
@@ -94,29 +91,23 @@ export function TaskList({ onSelectTask, onNewTask }: TaskListInternalProps) {
     [filteredTasks, moveTask],
   );
 
-  const handleDragStart = useCallback(
-    (e: React.DragEvent, taskId: string) => {
-      setDraggedTaskIdLocal(taskId);
-      e.dataTransfer.effectAllowed = "move";
-      e.dataTransfer.setData("text/plain", taskId);
-    },
-    [],
-  );
+  const handleDragStart = useCallback((e: React.DragEvent, taskId: string) => {
+    setDraggedTaskIdLocal(taskId);
+    e.dataTransfer.effectAllowed = "move";
+    e.dataTransfer.setData("text/plain", taskId);
+  }, []);
 
-  const handleDragOver = useCallback(
-    (e: React.DragEvent, index: number) => {
-      e.preventDefault();
-      e.dataTransfer.dropEffect = "move";
+  const handleDragOver = useCallback((e: React.DragEvent, index: number) => {
+    e.preventDefault();
+    e.dataTransfer.dropEffect = "move";
 
-      const rect = e.currentTarget.getBoundingClientRect();
-      const midpoint = rect.top + rect.height / 2;
-      const mouseY = e.clientY;
+    const rect = e.currentTarget.getBoundingClientRect();
+    const midpoint = rect.top + rect.height / 2;
+    const mouseY = e.clientY;
 
-      setDragOverIndexLocal(index);
-      setDropPositionLocal(mouseY < midpoint ? "top" : "bottom");
-    },
-    [],
-  );
+    setDragOverIndexLocal(index);
+    setDropPositionLocal(mouseY < midpoint ? "top" : "bottom");
+  }, []);
 
   const handleDragLeave = useCallback(() => {
     setDragOverIndexLocal(null);
@@ -163,7 +154,6 @@ export function TaskList({ onSelectTask, onNewTask }: TaskListInternalProps) {
     setDropPositionLocal(null);
   }, []);
 
-
   useEffect(() => {
     setStatusBar({
       statusText: `${filteredTasks.length} task${filteredTasks.length === 1 ? "" : "s"}`,