Issue metadata
Sign in to add a comment
|
Android text composition underlines are buggy when tapping between words |
||||||||||||||||||||||||
Issue descriptionChrome Version: 55.0.2883.91 OS: Android 7.1.2 What steps will reproduce the problem? (1) Go to editpad.org (2) Type some text (several words separated by spaces) (3) Tap back and forth between the words What is the expected result? The black underline marking the composition range should always be correctly positioned under a word. What happens instead? The underline is kind of jumpy and will go through an intermediate state where it seems to have the correct length for the final underline but will be positioned incorrectly.
,
Jan 24 2017
There's a pending patch https://codereview.chromium.org/2370663002/ removing those restartInput calls, I'm guessing that probably fixes it?
,
Jan 24 2017
Yes, I've confirmed that that patch will fix this issue.
,
Jan 24 2017
,
Feb 3 2017
I have some doubt that removing restartInput on selection change can completely fix the issue because there are other events that can lead to restartinput, such as focus change and cancel composition. Also, at the time IME redesign landed on M54, I did not see this. I suspect that Blink change caused this, and it may help understand what happened. Therefore requesting a full bisect.
,
Feb 10 2017
I think b/35198308 and b/34961224 are likely caused by this issue, so raising priority.
,
Feb 10 2017
Ryan's not an expert on this issue, so assigning to you if you think it's worth investigating further. I suspect removing cursor restartInput probably helped with the majority of the cases so the severity might at least be lower as of M58.
,
Feb 14 2017
Hmm... This is still happening on ToT with the following scenario: 1) Go to textarea.org 2) Type 'Abcde' 3) Type enter 4) Type 'Abcde' again However, it does not happen on stable version (55.0.2883.91). I suspect that Google Keyboard recognizes that it's Chrome stable and do some workaround. Or it could be Google Keyboard's own issue. Let me look into it.
,
Feb 14 2017
You can test the theory about Google Keyboard checking for Chrome Stable by installing a new ChromeStable.apk build from https://pantheon.corp.google.com/storage/browser/chrome-signed/android-B0urB0N/, and seeing if the same version ChromeCanary.apk behaves different.
,
Feb 14 2017
,
Feb 15 2017
I could reproduce issue described in bug description all the way back to M51 if I use multiple lines of text. The issue in comment 9 is due to: https://codereview.chromium.org/2568093003
,
Feb 15 2017
I split off the bug in #9 into http://crbug.com/692699 for clarity. Reserving this issue for preexisting problem.
,
Feb 22 2017
Confirmed that https://codereview.chromium.org/2370663002 fixed this issue as well. Marking as dup because it has the same root cause.
,
Feb 22 2017
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by rlanday@chromium.org
, Jan 24 2017