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: A symptom of characters being copied when there are multiple TextEdits. #4374
base: master
Are you sure you want to change the base?
Conversation
@@ -44,6 +44,10 @@ pub struct TextEditState { | |||
#[cfg_attr(feature = "serde", serde(skip))] | |||
pub(crate) has_ime: bool, | |||
|
|||
// target ID for IME candidate. | |||
#[cfg_attr(feature = "serde", serde(skip))] | |||
pub(crate) ime_target_id: Id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't this always be the same Id
as the key used to find TextEditState
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, it only stores TextEdit ID.
It's currently working fine without any problems.
If something is wrong or needs to be fixed, please fix it or let me know.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That the code is working fine is not enough. I need to maintain this, and so I need to understand it. Storing the Id
here makes no sense to me, since it is already the key.
Blocked on #4358 |
for example, Please review. |
Event::Ime
#4358Fix: A symptom of characters being copied when there are multiple TextEdits.