Update useKeyboard hook to use default argument for `shouldHandleKe…

…yboardEvents` (#265) * update useKeyboard argument shouldHandleKeyboardEvents to default to true * Add changesets patch bump for useKeyboard hook
FormidableLabs · Oct 12, 2024 · ed9b693 · ed9b693
1 parent 95e3cae
commit ed9b693
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/.changeset/eighty-planets-think.md b/.changeset/eighty-planets-think.md
@@ -0,0 +1,5 @@
+---
+'@react-native-ama/extras': patch
+---
+
+Updated useKeyboard hook argument shouldHandleKeyboardEvents to default to true and not be required.
diff --git a/packages/extras/src/hooks/useKeyboard.ts b/packages/extras/src/hooks/useKeyboard.ts
@@ -21,19 +21,19 @@ const KEYBOARD_EVENT_HIDE: KeyboardEventName = Platform.select({
   default: 'keyboardDidHide',
 });
 
-export const useKeyboard = (shouldHandleKeyboardEvents: boolean) => {
+export const useKeyboard = (shouldHandleKeyboardEvents: boolean = true) => {
   const keyboardHeight = useSharedValue(0);
   const keyboardFinalHeight = useSharedValue(0);
   const isKeyboardVisible = useSharedValue(false);
 
   const handleKeyboardEvent = useCallback(
     (
       isVisible: boolean,
-      height: number,
+      endCoordinatesHeight: number,
       duration: number,
       easing: KeyboardEventEasing,
     ) => {
-      const finalHeight = isVisible ? height : 0;
+      const finalHeight = isVisible ? endCoordinatesHeight : 0;
 
       const animationConfig = getKeyboardAnimationConfigs(easing, duration);