Skip to content
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 iOS TextField crash when using speech-to-text #1183

Merged
merged 3 commits into from Mar 12, 2024

Conversation

ASalavei
Copy link
Collaborator

@ASalavei ASalavei commented Mar 11, 2024

Proposed Changes

Use iOS native implementation of UITextInputTokenizerProtocol represented by the class UITextInputStringTokenizer.

The PR reverts changes of #808 , however due to textInput = this parameter, the Fast Delete feature keeps working as intended.

Fixes: JetBrains/compose-multiplatform#4443

@igordmn igordmn removed their request for review March 11, 2024 18:26
@ASalavei
Copy link
Collaborator Author

Thanks @mazunin-v-jb we found a bug in characters range calculation. Also fixed in current PR.

Copy link
Member

@MatkovIvan MatkovIvan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The general approach to use native implementation LGTM

Needs a second approval - I didn't test it on a real device

@ASalavei ASalavei merged commit 8575810 into jb-main Mar 12, 2024
6 checks passed
@ASalavei ASalavei deleted the andrei.salavei/fix-voice-input-crash branch March 12, 2024 13:01
ASalavei added a commit that referenced this pull request Mar 12, 2024
## Proposed Changes
Use iOS native implementation of `UITextInputTokenizerProtocol`
represented by the class `UITextInputStringTokenizer`.

The PR reverts changes of
#808 ,
however due to `textInput = this` parameter, the Fast Delete feature
keeps working as intended.

Fixes: JetBrains/compose-multiplatform#4443
@ASalavei ASalavei changed the title Use ios native implementation of UITextInputTokenizerProtocol Fix iOS TextField crash when using speech-to-text Mar 13, 2024
@ASalavei ASalavei added the changelog1.6.1 Temporary label for 1.6.1 changelog (will be removed) label Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog1.6.1 Temporary label for 1.6.1 changelog (will be removed)
Projects
None yet
3 participants