RTL CompositionUnderline underline is painted incorrectly. |
|||||
Issue descriptionBreakout from https://codereview.chromium.org/2337373002/ Most of the RTL test cases (see expectation images in above) have a broken underline. Probably just a logic bug as there were AFAIK no tests or easy way to scrutinize visually until now. I'm assuming we still want to keep the RTL code path, I don't personally know that the IME (which I believe is only real user of this code) is needed for RTL languages, but I would think that it is. If someone knows otherwise speak up as we could instead delete the RTL logic.
,
Sep 23 2016
Roozbeh, could you help here? Are there cases for where IMEs are needed for RTL languages?
,
Oct 20 2016
Note I am in discussion with samsmlee@ on internal bug 32237911 currently which involves Hebrew text and use of Google Keyboard on Android and likely relates to the original bug http://crbug.com/641261 So, this loosely corroborates that we do need RTL IME/Composition Underlines.
,
May 11 2017
Unassigning self from bugs that I don't expect to be able to get to soon in case someone else is able to pick them up.
,
May 14 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 14 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by drott@chromium.org
, Sep 21 2016