-
-
Notifications
You must be signed in to change notification settings - Fork 639
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
SmartUrlInput: Remove workaround for an RN bug that's less bothersome…
… now Less bothersome because, after the previous commit, the only remaining .focus() call is for convenience (autofocus when back-navigating to the screen), not necessity. With this RN bug, whether we work around it or not, it remains possible to focus a TextInput by tapping on it. So why "necessity"? Before the previous commit, we had a hack where text elements were trying to blend in as part of a TextInput, with the real TextInput being as little as 1px wide when empty. So, pressing the text elements was an important way to give the input focus, and that was achieved with .focus().
- Loading branch information
1 parent
9bdac88
commit 9303039
Showing
1 changed file
with
1 addition
and
56 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters