Closed
Bug 761953
Opened 12 years ago
Closed 12 years ago
Prevent the b2g vkd from being hidden when browsing Google.com
Categories
(Firefox OS Graveyard :: General, defect)
Firefox OS Graveyard
General
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: timdream, Unassigned)
Details
Attachments
(2 files)
1.76 KB,
patch
|
Details | Diff | Splinter Review | |
2.03 KB,
patch
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #755648 +++ Gaia issue: https://github.com/mozilla-b2g/gaia/issues/1321 The root cause of this behavior is because for some unknown reason the VKB Manager keeps getting trusted keypress event with escape key when google.com is loaded in the browser. This patch will have VKB manager listens to keyup for hiding keyboard instead. Also, the keypress and keydown fired before keyup will be preventDefault and stopPropagation'ed.
Attachment #630475 -
Flags: review?(21)
Comment 1•12 years ago
|
||
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TW) from comment #0) > The root cause of this behavior is because for some unknown reason the VKB > Manager keeps getting trusted keypress event with escape key when google.com > is loaded in the browser. Maybe you should investigate this instead of trying to work around the issue?
Reporter | ||
Comment 2•12 years ago
|
||
Patch to commit if r+
Reporter | ||
Comment 3•12 years ago
|
||
@vingtetun, feedback?
Comment 4•12 years ago
|
||
(In reply to Tim Guan-tin Chien [:timdream] (MoCo-TW) from comment #3) > @vingtetun, feedback? I would like to understand the root cause of this behavior before changing from keypress to keydown/keyup.
Comment 5•12 years ago
|
||
I can't reproduce the issue on a desktop build.
Comment 6•12 years ago
|
||
Comment on attachment 630475 [details] [diff] [review] Patch v1 Since I can't reproduce and the browser is broken because of 'remote', I remote the r? flag.
Attachment #630475 -
Flags: review?(21)
Reporter | ||
Comment 7•12 years ago
|
||
Doesn't happen anymore since mozKeyboard API had changed.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•