added useCallback because usage of autoSelect

abusix · Nov 7, 2023 · 7743db8 · 7743db8
1 parent 2139df7
commit 7743db8
Showing 1 changed file with 3 additions and 7 deletions.
diff --git a/src/components/form-field/search-input/search-input.tsx b/src/components/form-field/search-input/search-input.tsx
@@ -1,4 +1,4 @@
-import React, { useMemo, useRef } from "react";
+import React, { useCallback, useRef } from "react";
 import { classNames } from "../../../util/class-names";
 import CrossIcon from "../../../icons/cross-icon";
 import SearchIcon from "../../../icons/search-icon";
@@ -24,15 +24,11 @@ export const SearchInput = ({
     const inputRef = useRef<HTMLInputElement>(null);
     const isClearIconShown = !readOnly && !disabled && value !== undefined && value !== "";
 
-    const isClearIconShown = useMemo(() => {
-        return !readOnly && !disabled && value?.toString().length;
-    }, [disabled, readOnly, value]);
-
-    const handleAutoSelection = () => {
+    const handleAutoSelection = useCallback(() => {
         if (autoSelect && inputRef.current) {
             inputRef.current.select();
         }
-    };
+    }, [autoSelect]);
 
     return (
         <div className={classNames("relative w-full")}>