Skip to content

fix: TextInput onTextChanged triggered incorrectly #31376

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

Closed
wants to merge 1 commit into from

Conversation

rnike
Copy link
Contributor

@rnike rnike commented Apr 17, 2021

Summary

fix #30578

Using the language like Chinese, we need to construct a character with symbols or alphabets, the onTextChanged from TextInput is not handling this kind of language correctly.

Changelog

[iOS] [Fixed] - TextInput onTextChanged triggered incorrectly

Test Plan

Before

before.mov

After

after_applied.mov

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 17, 2021
@rnike rnike changed the title fix(#30578): TextInput onTextChanged triggered incorrectly fix: TextInput onTextChanged triggered incorrectly Apr 17, 2021
@rnike rnike marked this pull request as ready for review April 17, 2021 21:56
@analysis-bot
Copy link

analysis-bot commented Apr 17, 2021

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 91452fc
Branch: main

@analysis-bot
Copy link

analysis-bot commented Apr 17, 2021

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,788,244 +0
android hermes armeabi-v7a 7,191,825 +0
android hermes x86 8,099,598 +0
android hermes x86_64 8,077,103 +0
android jsc arm64-v8a 9,656,305 +0
android jsc armeabi-v7a 8,428,809 +0
android jsc x86 9,608,478 +0
android jsc x86_64 10,203,027 +0

Base commit: 91452fc
Branch: main

@xilin
Copy link

xilin commented May 18, 2021

It is amazing that we still cannot get markedTextRange in RN :(

@rnike rnike force-pushed the fix/30578/input-onchanged branch from e6a6349 to ce4c42a Compare June 6, 2021 16:40
@ouabing
Copy link

ouabing commented Jun 15, 2021

Hoping this fix can be included in next release.

@react-native-bot react-native-bot added Bug Platform: iOS iOS applications. labels Oct 1, 2021
@rnike rnike force-pushed the fix/30578/input-onchanged branch from ce4c42a to b67c212 Compare December 27, 2021 03:59
@rnike
Copy link
Contributor Author

rnike commented Dec 27, 2021

rebase to main for ci, but somehow the ci still failed

@rnike rnike force-pushed the fix/30578/input-onchanged branch from b67c212 to ea7cbb9 Compare May 31, 2022 18:27
@KingAmo
Copy link

KingAmo commented Oct 8, 2022

any updates on this PR?

Copy link

This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions bot added the Stale There has been a lack of activity on this issue and it may be closed soon. label Dec 30, 2023
Copy link

github-actions bot commented Jan 7, 2024

This PR was closed because it has been stalled for 7 days with no activity.

@github-actions github-actions bot closed this Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Platform: iOS iOS applications. Stale There has been a lack of activity on this issue and it may be closed soon.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TextInput onChangeText is firing at wrong time when type with Chinese input keyboard in iOS
8 participants