Skip to content

Conversation

lotem
Copy link
Member

@lotem lotem commented Mar 2, 2024

call Context::BeginEditing when input is edited after selection. move cursor also counts as edit.

the tag is used to tell if BackSpace should delete the last input character or revert a recent selection.

the information was previously denoted by kConfirmed status which caused the side-effect of breaking a user phrase into individual segments.

fixed #746, fixed #830

@lotem lotem self-assigned this Mar 2, 2024
@lotem lotem requested a review from ksqsf March 2, 2024 05:58
call Context::BeginEditing when input is edited after selection.
move cursor also counts as edit.

the tag is used to tell if BackSpace should delete the last input
character or revert a recent selection.

the information was previously denoted by kConfirmed status which caused
the side-effect of breaking a user phrase into individual segments.

fixed rime#746, fixed rime#830
Copy link
Member

@ksqsf ksqsf left a comment

Choose a reason for hiding this comment

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

Verified to work. LGTM

@lotem lotem merged commit 8b7f6b7 into rime:master Mar 2, 2024
@lotem lotem deleted the fix branch March 5, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants