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

Batch frequent Marked Text edit commands #1692

Merged
merged 4 commits into from
Nov 29, 2024

Conversation

ASalavei
Copy link
Collaborator

Add code that joins several frequent text edit commands into a single commands batch.

Fixes https://youtrack.jetbrains.com/issue/CMP-6786/when-use-Chinese-keyboard-inputs-get-interrupted.

Release Notes

Fixes - iOS

  • Fixes an interruption while typing characters on a Chinese keyboard

@@ -116,23 +125,4 @@ internal interface IOSSkikoInput {
* Returned value must be in range between 0 and length of text (inclusive).
*/
fun positionFromPosition(position: Long, offset: Long): Long

object Empty : IOSSkikoInput {

Choose a reason for hiding this comment

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

Is it not used anymore?

Copy link
Collaborator Author

@ASalavei ASalavei Nov 28, 2024

Choose a reason for hiding this comment

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

Yes, It hasn't beed used before as well.

@ASalavei ASalavei requested a review from igordmn November 29, 2024 15:27
@ASalavei ASalavei merged commit 4fb08ca into jb-main Nov 29, 2024
6 of 7 checks passed
@ASalavei ASalavei deleted the andrei.salavei/batch-some-update-commands branch November 29, 2024 17:34
ASalavei added a commit that referenced this pull request Dec 9, 2024
Add code that joins several frequent text edit commands into a single
commands batch.

Fixes
https://youtrack.jetbrains.com/issue/CMP-6786/when-use-Chinese-keyboard-inputs-get-interrupted.

## Release Notes
### Fixes - iOS
- Fixes an interruption while typing characters on a Chinese keyboard
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.

3 participants