Last modified: 2014-09-04 00:14:40 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T69262, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 67262 - VisualEditor Mobile: Chrome for mobile keyboard doesn't fire useful key events for backspace
VisualEditor Mobile: Chrome for mobile keyboard doesn't fire useful key event...
Status: ASSIGNED
Product: VisualEditor
Classification: Unclassified
Mobile (Other open bugs)
unspecified
All All
: High major
: ---
Assigned To: Editing team bugs – take if you're interested!
:
Depends on:
Blocks: ve-androidsupport
  Show dependency treegraph
 
Reported: 2014-06-28 22:31 UTC by Ed Sanders
Modified: 2014-09-04 00:14 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Ed Sanders 2014-06-28 22:31:33 UTC
Key events fire without key codes (understandable as you can 'swipe' whole words) and backspace fires no key[up|down|press] event at all.
Comment 1 James Forrester 2014-06-28 22:32:23 UTC
Oy.
Comment 2 Ed Sanders 2014-06-28 22:44:06 UTC
There is an 'input' event which fires, but this doesn't solve the problem of prevent the default backspace behaviour.
Comment 3 Ed Sanders 2014-06-28 22:44:59 UTC
See https://code.google.com/p/chromium/issues/detail?id=118639 for pitchfork waving, including a complaint from the Microsoft Office Web Apps team who are having the same problem.
Comment 4 Ed Sanders 2014-06-28 23:03:13 UTC
Not quite true, see http://jsbin.com/alAKaNaZ/5

Backspace fires a keydown with keyCode 0 (same as every other key event) but not when whitespace is deleted.

See https://code.google.com/p/chromium/issues/detail?id=184812
Comment 5 Ed Sanders 2014-06-28 23:04:19 UTC
"No key event is generated when the BACKSPACE key is pressed outside of composition, in other words, when AdapterInputConnection.deleteSurroundingText is called. In the reported bug, since the text field is empty, thus IME is not in the composition mode, thus pressing the BACKSPACE key triggers deleteSurroundingText and thus no key event is generated.

A cl that generates key events in deleteSurroundingText with a dummy keycode is under review at https://codereview.chromium.org/348413003/."
Comment 6 Ed Sanders 2014-07-02 22:28:31 UTC
It looks like the above commit to Chrome for Mobile has been merged, which solves the most serious case (no event fired at all). We are still left with the general problem of IME deletion.
Comment 7 James Forrester 2014-07-03 02:06:44 UTC
(In reply to Ed Sanders from comment #6)
> It looks like the above commit to Chrome for Mobile has been merged, which
> solves the most serious case (no event fired at all).

It doesn't "solve" it perfectly, as OS update lag for Android devices is notoriously slow (or non existent), and app update take-up is apparently poor. Is there a sane work-around we could do?

> We are still left with the general problem of IME deletion.

Yeah. :-(
Comment 8 Ed Sanders 2014-07-03 10:02:58 UTC
OS update is not an issue, just app.
Comment 9 Brion Vibber 2014-07-03 15:16:20 UTC
Note that Android 4.4's WebView is based on the Chromium engine, and unlike the standalone Chrome app it only gets updated with the OS. Might want to check whether the  stock Android browser on non-Google-branded 4.4.x devices is also affected. (Nexus devices seem to leave out the AOSP default browser in favor of Chrome, but I see a customized version on Samsung Galaxy branded devices for instance.)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links