Closed
Bug 778488
Opened 12 years ago
Closed 12 years ago
Reader Mode: Cannot pan down when finger is over an image
Categories
(Firefox for Android Graveyard :: Reader View, defect)
Tracking
(firefox16 verified, firefox17 verified, firefox18 verified, fennec16+)
People
(Reporter: pretzer, Assigned: lucasr)
References
Details
In reader mode it is not possible to pan down when the finger is over an image. Panning upwards is possible.
STR:
1) Visit http://m.wired.com/gadgetlab/2012/07/apple-reveals-for-monday-trial/
2) Try to pan down with the finger over the image at the top
Expected: Site pans down
Actual: Nothing happens
Comment 1•12 years ago
|
||
Is this reader-mode specific? CC'ing :cats
Reporter | ||
Comment 2•12 years ago
|
||
(In reply to Aaron Train [:aaronmt] from comment #1)
> Is this reader-mode specific? CC'ing :cats
Yes, I can pan around fine on the original article.
Comment 3•12 years ago
|
||
The readability code appears to be mangling the page somehow so that the <a> tag wrapping the image has overflow: scroll set on it. Also for some reason the clientWidth/clientHeight of the <a> tag are 0, while the scrollWidth/scrollHeight are non-zero. This makes us detect that <a> tag as scrollable, and so we intercept all of those touch events and try to use them for scrolling the <a> tag, which isn't actually scrollable.
Reporter | ||
Updated•12 years ago
|
Updated•12 years ago
|
tracking-fennec: ? → 16+
Assignee | ||
Comment 5•12 years ago
|
||
This is a bug in reader's CSS. I have a fix for it.
Assignee: nobody → lucasr.at.mozilla
Comment 7•12 years ago
|
||
When I attempt to pan, my console floods with: I/GeckoSubdocumentScrollHelper(10060): Got message: Gesture:ScrollAck
Using the URL: http://tantek.com/2012/222/b1/cole-valley-cell-tower-approved-planning-commission
Assignee | ||
Comment 8•12 years ago
|
||
This will be fixed as part of bug 766948.
Assignee | ||
Comment 9•12 years ago
|
||
This is now fixed in m-c with patch: https://hg.mozilla.org/mozilla-central/rev/45b6b015f792
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 11•12 years ago
|
||
The fix was pushed to mozilla-aurora as stated in bug 766948:
https://hg.mozilla.org/releases/mozilla-aurora/rev/f8bb07e34140
Updated•12 years ago
|
status-firefox18:
--- → verified
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
•