react-native-keyboard-controller 1.9.1 → 1.9.3

package/android/src/main/java/com/reactnativekeyboardcontroller/extensions/View.kt

@@ -2,6 +2,7 @@ package com.reactnativekeyboardcontroller.extensions
 
 import android.graphics.Rect
 import android.os.Build
+import android.util.Log
 import android.view.View
 import androidx.annotation.RequiresApi
 
@@ -37,15 +38,12 @@ private val tmpIntArr = IntArray(2)
  */
 @RequiresApi(Build.VERSION_CODES.KITKAT)
 fun View.copyBoundsInWindow(rect: Rect) {
-  if (isLaidOut && isAttachedToWindow) {
+  if (isAttachedToWindow) {
     rect.set(0, 0, width, height)
     getLocationInWindow(tmpIntArr)
     rect.offset(tmpIntArr[0], tmpIntArr[1])
   } else {
-    throw IllegalArgumentException(
-      "Can not copy bounds as view is not laid out" +
-        " or attached to window",
-    )
+    Log.w("View.copyBoundsInWindow", "Can not copy bounds as view is not attached to window")
   }
 }
 

package/ios/observers/FocusedInputLayoutObserver.swift

@@ -25,6 +25,8 @@ let noFocusedInputEvent: [String: Any] = [
 public class FocusedInputLayoutObserver: NSObject {
   // class members
   var onEvent: (NSDictionary) -> Void
+  // state variables
+  private var isMounted = false
   // input tracking
   private var currentInput: UIView?
   private var hasKVObserver = false
@@ -37,6 +39,12 @@ public class FocusedInputLayoutObserver: NSObject {
   }
 
   @objc public func mount() {
+    if isMounted {
+      return
+    }
+
+    isMounted = true
+
     NotificationCenter.default.addObserver(
       self,
       selector: #selector(keyboardWillShow),
@@ -52,6 +60,7 @@ public class FocusedInputLayoutObserver: NSObject {
   }
 
   @objc public func unmount() {
+    isMounted = false
     // swiftlint:disable:next notification_center_detachment
     NotificationCenter.default.removeObserver(self)
   }

package/ios/observers/KeyboardMovementObserver.swift

@@ -31,6 +31,7 @@ public class KeyboardMovementObserver: NSObject {
   private var prevKeyboardPosition = 0.0
   private var displayLink: CADisplayLink?
   private var hasKVObserver = false
+  private var isMounted = false
   // state variables
   private var keyboardHeight: CGFloat = 0.0
   private var duration = 0
@@ -45,6 +46,12 @@ public class KeyboardMovementObserver: NSObject {
   }
 
   @objc public func mount() {
+    if isMounted {
+      return
+    }
+
+    isMounted = true
+
     NotificationCenter.default.addObserver(
       self,
       selector: #selector(keyboardWillDisappear),
@@ -138,6 +145,7 @@ public class KeyboardMovementObserver: NSObject {
   }
 
   @objc public func unmount() {
+    isMounted = false
     // swiftlint:disable:next notification_center_detachment
     NotificationCenter.default.removeObserver(self)
   }

package/ios/views/KeyboardControllerView.mm

@@ -156,17 +156,36 @@ using namespace facebook::react;
 
   if (newViewProps.enabled != oldViewProps.enabled) {
     if (newViewProps.enabled) {
-      [inputObserver mount];
-      [keyboardObserver mount];
+      [self mount];
     } else {
-      [inputObserver unmount];
-      [keyboardObserver unmount];
+      [self unmount];
     }
   }
 
   [super updateProps:props oldProps:oldProps];
 }
 
+- (void)willMoveToSuperview:(UIView *)newSuperview
+{
+  if (newSuperview == nil) {
+    [self unmount];
+  } else {
+    [self mount];
+  }
+}
+
+- (void)mount
+{
+  [inputObserver mount];
+  [keyboardObserver mount];
+}
+
+- (void)unmount
+{
+  [inputObserver unmount];
+  [keyboardObserver unmount];
+}
+
 Class<RCTComponentViewProtocol> KeyboardControllerViewCls(void)
 {
   return KeyboardControllerView.class;

package/ios/views/KeyboardControllerViewManager.swift

@@ -24,11 +24,9 @@ class KeyboardControllerView: UIView {
   @objc var enabled: ObjCBool = true {
     didSet {
       if enabled.boolValue {
-        inputObserver?.mount()
-        keyboardObserver?.mount()
+        mount()
       } else {
-        inputObserver?.unmount()
-        keyboardObserver?.unmount()
+        unmount()
       }
     }
   }
@@ -37,6 +35,8 @@ class KeyboardControllerView: UIView {
     self.bridge = bridge
     eventDispatcher = bridge.eventDispatcher()
     super.init(frame: frame)
+    inputObserver = FocusedInputLayoutObserver(handler: onInput)
+    keyboardObserver = KeyboardMovementObserver(handler: onEvent, onNotify: onNotify)
   }
 
   @available(*, unavailable)
@@ -44,21 +44,17 @@ class KeyboardControllerView: UIView {
     fatalError("init(coder:) has not been implemented")
   }
 
-  override func willMove(toWindow newWindow: UIWindow?) {
-    if newWindow == nil {
-      // Will be removed from a window
-      inputObserver?.unmount()
-      keyboardObserver?.unmount()
-    }
-  }
+  // for mounting/unmounting observers for lifecycle events we're using willMove(toSuperview) method
+  // not willMove(toWindow)
+  // see https://github.com/kirillzyusko/react-native-keyboard-controller/issues/271
+  override func willMove(toSuperview newSuperview: UIView?) {
+    super.willMove(toSuperview: newSuperview)
 
-  override func didMoveToWindow() {
-    if window != nil {
-      // Added to a window
-      inputObserver = FocusedInputLayoutObserver(handler: onInput)
-      inputObserver?.mount()
-      keyboardObserver = KeyboardMovementObserver(handler: onEvent, onNotify: onNotify)
-      keyboardObserver?.mount()
+    if newSuperview == nil {
+      // The view is about to be removed from its superview (destroyed)
+      unmount()
+    } else {
+      mount()
     }
   }
 
@@ -90,4 +86,14 @@ class KeyboardControllerView: UIView {
   func onNotify(event: String, data: Any) {
     KeyboardController.shared()?.sendEvent(event, body: data)
   }
+
+  private func mount() {
+    inputObserver?.mount()
+    keyboardObserver?.mount()
+  }
+
+  private func unmount() {
+    inputObserver?.unmount()
+    keyboardObserver?.unmount()
+  }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "react-native-keyboard-controller",
-  "version": "1.9.1",
+  "version": "1.9.3",
   "description": "Keyboard manager which works in identical way on both iOS and Android",
   "main": "lib/commonjs/index",
   "module": "lib/module/index",