[react-bootstrap#4194] - Remove extra handlers in favor of usestate

zhuber · Oct 12, 2019 · 4f6aed9 · 4f6aed9
1 parent b6de548
commit 4f6aed9
Showing 1 changed file with 5 additions and 9 deletions.
diff --git a/src/ToggleButton.js b/src/ToggleButton.js
@@ -65,13 +65,9 @@ const ToggleButton = React.forwardRef(
   ) => {
     const [focused, setFocused] = useState(false);
 
-    const handleFocus = e => {
-      if (e.target.tagName === 'INPUT') setFocused(true);
-    };
-
-    const handleBlur = e => {
-      if (e.target.tagName === 'INPUT') setFocused(false);
-    };
+    const toggleFocus = useCallback(e => {
+      if (e.target.tagName === 'INPUT') setFocused(!focused);
+    }, []);
 
     return (
       <Button
@@ -94,8 +90,8 @@ const ToggleButton = React.forwardRef(
           autoComplete="off"
           checked={!!checked}
           disabled={!!disabled}
-          onFocus={handleFocus}
-          onBlur={handleBlur}
+          onFocus={toggleFocus}
+          onBlur={toggleFocus}
           onChange={onChange || noop}
         />