Skip to content

When the user moves the caret during a composition, commit the composition #2255

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

Merged

Conversation

m-sasha
Copy link
Member

@m-sasha m-sasha commented Jul 17, 2025

When the user moves the caret during a composition, e.g. by clicking somewhere, commit the composition immediately.

Fixes https://youtrack.jetbrains.com/issue/CMP-8200

Testing

Tested manually.
Unfortunately it's not possible to create a unit test because the fix implies bidirectional interaction with the InputContext.

This should be tested by QA

Release Notes

Fixes - Desktop

  • [TextField] Fixed duplication of the composed characters when moving the caret by clicking during a composition.

@m-sasha m-sasha force-pushed the m-sasha/commit-composition-on-unexpected-caret-movement branch from ece5b64 to 6f37fe7 Compare July 17, 2025 15:29
@m-sasha m-sasha requested a review from igordmn July 18, 2025 09:24
@m-sasha m-sasha force-pushed the m-sasha/commit-composition-on-unexpected-caret-movement branch from 6f37fe7 to 41abdd8 Compare July 20, 2025 14:35
@m-sasha m-sasha requested a review from igordmn July 20, 2025 14:36
@m-sasha m-sasha merged commit a099758 into jb-main Jul 21, 2025
10 checks passed
@m-sasha m-sasha deleted the m-sasha/commit-composition-on-unexpected-caret-movement branch July 21, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants