fix(tasks): resolve comments

Author: Adinorio

apps/web/src/app/[locale]/(dashboard)/[wsId]/tasks/boards/[boardId]/_components/list-view.tsx

@@ -255,7 +255,12 @@ export function ListView({
         return updateObj;
       });
 
-      if (updates.some((obj) => Object.keys(obj).length > 1)) {
+      // Check if any updates have meaningful changes (more than just the id field)
+      const hasUpdates = updates.some((obj) => 
+        obj.priority !== undefined || obj.archived !== undefined || obj.tags !== undefined
+      );
+      
+      if (hasUpdates) {
         // Only call RPC if at least one field is being updated
         const { error } = await supabase.rpc('update_many_tasks', { updates });
         if (error) throw error;

apps/web/src/app/[locale]/(dashboard)/[wsId]/tasks/boards/[boardId]/kanban.tsx

@@ -140,11 +140,23 @@ export function KanbanBoard({ boardId, tasks, isLoading }: Props) {
       const task = active.data.current.task;
       setActiveTask(task);
       pickedUpTaskColumn.current = String(task.list_id);
-      // Get the DOM node of the dragged card
-      const cardNode = document.querySelector(`[data-id="${task.id}"]`);
+      
+      // Use more specific selector for better reliability
+      // Prefer data-id selector over generic querySelector
+      const cardNode = document.querySelector(`[data-id="${task.id}"]`) as HTMLElement;
       if (cardNode) {
         const cardRect = cardNode.getBoundingClientRect();
         dragStartCardLeft.current = cardRect.left;
+      } else {
+        // Fallback: try to find the card by task ID in a more specific way
+        const taskCards = document.querySelectorAll('[data-id]');
+        const targetCard = Array.from(taskCards).find(
+          (card) => card.getAttribute('data-id') === task.id
+        ) as HTMLElement;
+        if (targetCard) {
+          const cardRect = targetCard.getBoundingClientRect();
+          dragStartCardLeft.current = cardRect.left;
+        }
       }
       return;
     }

apps/web/src/app/[locale]/(dashboard)/[wsId]/tasks/boards/[boardId]/task-list.tsx

@@ -740,6 +740,7 @@ export const BoardColumn = React.memo(function BoardColumn({
                                   if (!filters.assignees.has('all')) {
                                     newAssignees.add('all');
                                   }
+                                  // Ensure mutual exclusivity with 'unassigned'
                                   setFilters((prev) => ({
                                     ...prev,
                                     assignees: newAssignees,
@@ -764,6 +765,7 @@ export const BoardColumn = React.memo(function BoardColumn({
                                   if (!filters.assignees.has('unassigned')) {
                                     newAssignees.add('unassigned');
                                   }
+                                  // Ensure mutual exclusivity with 'all'
                                   setFilters((prev) => ({
                                     ...prev,
                                     assignees: newAssignees,
@@ -795,8 +797,9 @@ export const BoardColumn = React.memo(function BoardColumn({
                                     onSelect={() => {
                                       const newAssignees = new Set(filters.assignees);
                                       if (filters.assignees.has('all') || filters.assignees.has('unassigned')) {
-                                        // If 'all' or 'unassigned' is selected, start fresh
-                                        newAssignees.clear();
+                                        // If 'all' or 'unassigned' is selected, clear them to avoid conflicts
+                                        newAssignees.delete('all');
+                                        newAssignees.delete('unassigned');
                                       }
                                       if (isSelected) {
                                         newAssignees.delete(member.id);