-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AutoComplete] Fix onNewRequest not firing on iOS #4291
Conversation
@@ -325,7 +325,9 @@ class AutoComplete extends Component { | |||
|
|||
handleBlur = (event) => { | |||
if (this.state.focusTextField && this.timerTouchTapCloseId === null) { | |||
this.close(); | |||
setTimeout(() => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It appears that handleBlur is executed before handleItemTouchTap
Interesting! In that case, should we defer the all if
block into a setTimeout
?
Shouldn't we also clear the timeout during the unmounting?
53b828e
to
ea2538e
Compare
I couldn't reproduce this behavior on an iOS10.1 iPhone 6 simulator. |
I can confirm we are still seeing this behavior running our Cordova app on iphone6 with iOS v10.11. Same behavior with iOS v9.3.x Don't see this issue when running from iOS and accessing web browser version of our app using Safari mobile. |
Any progress on merging this PR? This happens on Cordova iOS. It is also worth to mention that this issue does not appear on longer item press. |
tests/ docs demo, and is linted.Closes #4290