Closed
Bug 719026
Opened 12 years ago
Closed 12 years ago
Blue background color remains for inputs, links etc, when panning inside iframes
Categories
(Firefox for Android Graveyard :: General, defect, P3)
Tracking
(fennec11+)
RESOLVED
FIXED
Firefox 16
Tracking | Status | |
---|---|---|
fennec | 11+ | --- |
People
(Reporter: martijn.martijn, Assigned: Margaret)
References
Details
(Keywords: testcase)
Attachments
(3 files)
See testcase, steps to reproduce: - Make a swipe movement over one of the iframes Expected result: - Content becomes (perhaps) for a short while blue, then is white again Actual result: - Content stays light blue (which happens when content like links, text inputs) are css :active when tapped upon. Tested on the LG Optimus Black on Android 2.2.2 in portrait mode. I guess this might be related to bug 719019.
Reporter | ||
Comment 1•12 years ago
|
||
This bug can also be seen in XUL Fennec, btw. Also, long tap -> Context Menu -> Paste, then the input in the iframe still stays blue.
Comment 2•12 years ago
|
||
Yeah, it does look related to bug 719019/bug 716863 in that some of the events go to the iframe and some do not.
Assignee: nobody → bugmail.mozilla
tracking-fennec: --- → ?
Comment 3•12 years ago
|
||
over to margaret assuming this is related to the iframe event issues she's been working on
Assignee: bugmail.mozilla → margaret.leibovic
tracking-fennec: ? → 11+
Priority: -- → P3
Assignee | ||
Comment 4•12 years ago
|
||
A lot has changed since this bug was filed, but I think this patch fixes the problem. In _cancelTapHighlight, we're trying to remove the active state on an element by setting the active state on the top-level document. However, this does not affect the active state set on elements in iframes. To fix that, we need to set the active state on that element's owner document. I found that with this change, it seems like we remove the highlight too quickly, maybe exposing a problem with how soon we call _cancelTapHighlight, but I'm not sure what the expected behavior is here.
Attachment #632718 -
Flags: review?(wjohnston)
Updated•12 years ago
|
Attachment #632718 -
Flags: review?(wjohnston) → review+
Assignee | ||
Comment 5•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/a2d50843d2a5
Target Milestone: --- → Firefox 16
Comment 6•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/a2d50843d2a5
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•3 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
•