6adba409e6
Summary: This implement the autoFocus functionality natively instead of calling the focus method from JS on mount. This is needed to properly fix the issue described in https://github.com/facebook/react-native/issues/27217, where when using native navigation (UINavigationController) text input focus needs to happen in the same frame transition starts or it leads to an animation bug in UIKit. My previous attempt fixed the problem only partially and the bug could still happen since there is no guaranty code executed in useEffect will end up in the same frame as the native view being created and attached. To fix this I added an autoFocus prop to the native component on iOS and in didAttachToWindow we focus the input if it is set. This makes sure the focus is set in the same frame as the view hierarchy containing the input is created. ## Changelog [iOS] [Fixed] - Add native support for TextInput autoFocus on iOS Pull Request resolved: https://github.com/facebook/react-native/pull/27803 Test Plan: - Tested that the UI glitch when transitionning to a screen with an input with autofocus no longer happens in my app. - Tested that autofocus still works in RNTester - Made sure that onFocus does get called and TextInputState is updated properly Differential Revision: D19673369 Pulled By: TheSavior fbshipit-source-id: 14d8486ac635901622ca667c0e61c75fb446e493 |
||
|---|---|---|
| .. | ||
| BaseText | ||
| RawText | ||
| Text | ||
| TextInput | ||
| VirtualText | ||
| RCTConvert+Text.h | ||
| RCTConvert+Text.m | ||
| RCTTextAttributes.h | ||
| RCTTextAttributes.m | ||
| RCTTextTransform.h | ||
| React-RCTText.podspec | ||
| Text.js | ||
| TextAncestor.js | ||
| TextProps.js | ||