Closed
Bug 1377749
Opened 7 years ago
Closed 7 years ago
Duplicate characters when changing the text case value on input event
Categories
(Firefox for Android Graveyard :: Keyboards and IME, defect)
Tracking
(firefox57 fixed)
RESOLVED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | fixed |
People
(Reporter: magic, Assigned: jchen)
Details
Attachments
(2 files)
225 bytes,
text/html
|
Details | |
4.74 KB,
patch
|
esawin
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.104 Safari/537.36
Steps to reproduce:
* Open the attachment
* Type 'a'
* Type 'b'
Actual results:
You get 'AAB'
Expected results:
You should get 'AB'
Reporter | ||
Comment 1•7 years ago
|
||
The bug seems occur when the value is transformed (with toUpperCase or a String.replace).
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → nchen
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: needinfo?(nchen)
Assignee | ||
Comment 3•7 years ago
|
||
Normally, we keep the Java composition when composition is committed on
the Gecko side. However, if we somehow lose the composition on the Java
side (e.g. due to script changing the text), we should reset the
keyboard so it's aware of the change.
Attachment #8897068 -
Flags: review?(esawin)
Updated•7 years ago
|
Attachment #8897068 -
Flags: review?(esawin) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91aeee278ee2
Reset keyboard when composition is lost; r=esawin
Comment 5•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(nchen)
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/59d0f9b5458b
Follow-up to add null check for editable; r=me
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(nchen)
Comment 7•7 years ago
|
||
bugherder |
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•