Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(context): tag selected segment after editing #831

Merged
merged 1 commit into from
Mar 2, 2024
Merged

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
9 checks passed
@lotem lotem deleted the fix branch March 5, 2024 15:45
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
2 participants