chore: useListBoxにhooksを統合したため、setterを依存関係に含める必要がなくなった

kufu · Jan 27, 2025 · 49ad100 · 49ad100
1 parent da61340
commit 49ad100
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/smarthr-ui/src/components/ComboBox/useListBox.tsx b/packages/smarthr-ui/src/components/ComboBox/useListBox.tsx
@@ -128,7 +128,7 @@ export const useListBox = <T,>({
     if (!isExpanded) {
       setActiveOption(null)
     }
-  }, [isExpanded, setActiveOption])
+  }, [isExpanded])
 
   const listBoxRef = useRef<HTMLDivElement>(null)
   const [listBoxRect, setListBoxRect] = useState<Rect>({
@@ -244,7 +244,7 @@ export const useListBox = <T,>({
         setActiveOption(null)
       }
     },
-    [activeOption, moveActiveOptionIndex, onAdd, onSelect, setActiveOption],
+    [activeOption, moveActiveOptionIndex, onAdd, onSelect],
   )
 
   const { createPortal } = usePortal()
@@ -281,7 +281,7 @@ export const useListBox = <T,>({
       setNavigationType('pointer')
       setActiveOption(option)
     },
-    [setActiveOption],
+    [],
   )
 
   const wrapperStyleAttr = useMemo(() => {