-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
Fix TextInput shadow props not update when setText on iOS #20634
Conversation
Generated by 🚫 dangerJS |
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.
hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
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.
This does not appear to be the root cause of the bug. Can you explain the reasoning behind adding this line?
It seems like the actual bug would originate in RCTBaseTextInputView.m.
@hramos Emm, I think it's not the cause from RCTBaseTextInputView.m. because we only call it when we compare it wether changed in shadow view. And because we don't update the |
What's the status of this PR? |
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.
_text
represents the actual prop value, changing that not from JS does not seem right to me.
Why does this change fix the issue?
I'll closing this because the issue already fixes. |
Fixes #20224
Test Plan:
The code as below, click
TouchableOpacity
, click textInput , edit text, then clickTouchableOpacity
again.Release Notes:
[IOS] [BUGFIX] [TextInput] - Fix TextInput shadow props not update when update local data.