-
-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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
Backspacing on Android with Samsung keyboard misplaces the caret #14707
Comments
These “reverse typing” issues are more widespread than I thought, we’re receiving reports of users every 1-2 days in the last few weeks. Is there something we can do to either mitigate the issue or to help you identify the root cause of it? |
Is it "reverse typing" after the deletion? I thought it's moving the caret by one, but when typing the letters, they will be in the correct order. |
I’m sorry, I confused the issues. There are actually three distinct issues on Android that we have received repeated reports for recently:
It’s sometimes a bit annoying to figure out what exactly is going wrong because these reports are mostly from users of our customers. This adds a lot of “latency” when asking for more information and the instructions to reproduce the issue are often imprecise. |
The work for this issue and others related is completed in the #16289, and we finalized the round of internal (successful) tests. I encourage everyone to test the PR if you have a chance. It should be merged in the following days, and will be a part of the next release. |
Fix (typing, engine): Predictive text should not get doubled while typing. Closes #16106. Fix (engine): The reverse typing effect should not happen after the focus change. Closes #14702. Thanks, @urbanspr1nter! Fix (engine, typing): Typing on Android should avoid modifying DOM while composing. Closes #13994. Closes #14707. Closes #13850. Closes #13693. Closes #14567. Closes: #11569.
Fix (typing, engine): Predictive text should not get doubled while typing. Closes ckeditor#16106. Fix (engine): The reverse typing effect should not happen after the focus change. Closes ckeditor#14702. Thanks, @urbanspr1nter! Fix (engine, typing): Typing on Android should avoid modifying DOM while composing. Closes ckeditor#13994. Closes ckeditor#14707. Closes ckeditor#13850. Closes ckeditor#13693. Closes ckeditor#14567. Closes: ckeditor#11569.
📝 Provide detailed reproduction steps (if any)
test
test
by hitting the backspace key once.Screen_Recording_20230727_083942_Chrome.mp4
✔️ Expected result
The trailing
t
should be removed with the caret being placed behindtes
.❌ Actual result
The trailing
t
is removed, but the caret advanced by another character and is now betweente
ands
. The editor will now look like this:te|s
where the|
denotes the caret position.❓ Possible solution
This sounds an awful lot like #14567 which is similar but no the same.
📃 Other details
If you'd like to see this fixed sooner, add a 👍 reaction to this post.
The text was updated successfully, but these errors were encountered: