refactor: improve task movement logic in TaskCard and remove unused u…

…seLocalStorage hook

src/components/TaskCard.tsx

@@ -11,11 +11,19 @@ const TaskCard = ({ status }: Props) => {
   const { moveTask, getTasksByStatus } = useTaskContext();
   const tasks = getTasksByStatus(status);
 
+  function handleMove(item: { task: TaskType; type: TaskStatus }) {
+    moveTask(item.task, status as TaskStatus);
+  }
+
   const [{ isOver }, dropRef] = useDrop(() => {
     return {
       accept: Object.values(TaskStatus),
-      drop(item: { task: TaskType; type: TaskStatus }) {
-        moveTask(item.task, status as TaskStatus);
+      drop(item: { task: TaskType; type: TaskStatus }, monitor) {
+        const didDrop = monitor.didDrop();
+        if (didDrop) {
+          return;
+        }
+        handleMove(item);
       },
       collect(monitor) {
         return {

src/contexts/task-context.tsx

@@ -1,6 +1,5 @@
 import { createContext, FC, ReactNode, useState } from "react";
 import { TaskCreateType, TaskStatus, TaskType } from "../types";
-import { useLocalStorage } from "../hooks";
 
 type TaskContextType = {
   tasks: TaskType[];
@@ -16,7 +15,7 @@ export const TaskContext = createContext<TaskContextType | undefined>(
 );
 
 export const TaskProvider: FC<{ children: ReactNode }> = ({ children }) => {
-  const [tasks, setTasks] = useLocalStorage<TaskType[]>("tasks", [
+  const [tasks, setTasks] = useState<TaskType[]>([
     {
       id: 1,
       name: "Task One",

src/hooks/index.tsx

@@ -1,4 +1,4 @@
-import { useContext, useEffect, useState } from "react";
+import { useContext } from "react";
 import { TaskContext } from "../contexts/task-context";
 
 export function useTaskContext() {
@@ -8,39 +8,3 @@ export function useTaskContext() {
   }
   return context;
 }
-
-export function useLocalStorage<T>(key: string, initialValue: T) {
-  const [storedValue, setStoredValue] = useState<T>(() => {
-    try {
-      const item = localStorage.getItem(key);
-      return item ? JSON.parse(item) : initialValue;
-    } catch (error) {
-      console.error("Error reading from localStorage:", error);
-      return initialValue;
-    }
-  });
-
-  const setValue = (value: T | ((val: T) => T)) => {
-    try {
-      const valueToStore =
-        value instanceof Function ? value(storedValue) : value;
-      setStoredValue(valueToStore);
-      localStorage.setItem(key, JSON.stringify(valueToStore));
-    } catch (error) {
-      console.error("Error writing to localStorage:", error);
-    }
-  };
-
-  useEffect(() => {
-    try {
-      const item = localStorage.getItem(key);
-      if (item) {
-        setStoredValue(JSON.parse(item));
-      }
-    } catch (error) {
-      console.error("Error reading from localStorage on key change:", error);
-    }
-  }, [key]);
-
-  return [storedValue, setValue] as const;
-}