Closed Bug 722657 Opened 10 years ago Closed 9 years ago
Panning problems with iframes
Tested on the LG Optimus Black, Android 2.2.2 in portrait mode. Steps to reproduce: - Do a fling action upwards to scroll down in the iframe Expected result: - Content inside iframe scrolls downward in reasonable speed Actual result: - Content inside iframe scrolls down much slower than expected Also, I noticed that swiping over the iframes doesn't really scroll the iframe content much, scrolling goes quite slow.
I made a video of it here: http://www.youtube.com/watch?v=6P1p6z7BcKU
10 years ago
Duplicate of this bug: 718144
Issue can be seen on reader.google.com
Duplicate of this bug: 730252
tracking-fennec: + → 15+
blocking-fennec1.0: ? → -
Seeing the same issue with standard frameset on, for example, http://www.cougarfan.com/article.asp?ArtID=75914&URL=http://www.sltrib.com/sltrib/blogsbyusports/53873304-65/byu-martineau-rose-coach.html.csp This page behaves almost exactly like http://di.se - (from bug 723945 - which was marked as a duplicate of this one)
While investigating some subdocument scrolling issues, I found a rather glaring bug that causes this issue. Will upload a patch shortly, it's an easy fix. Re-noming because I think this should block 1.0 - it happens on a range of websites and our subdocument scrolling behaviour really is quite atrocious.
Comment on attachment 620314 [details] [diff] [review] Save the pending displacement if we can't send it right away Review of attachment 620314 [details] [diff] [review]: ----------------------------------------------------------------- heh, well caught :) Looks good to me. ::: mobile/android/base/ui/SubdocumentScrollHelper.java @@ +60,5 @@ > > private boolean mOverridePanning; > private boolean mOverrideScrollAck; > private boolean mOverrideScrollPending; > private boolean mScrollSucceeded; I had a hard time reading this patch, I think it'd be good to add some documentation for these variables while you're changing code here.
Attachment #620314 - Flags: review?(chrislord.net) → review+
https://hg.mozilla.org/mozilla-central/rev/6483b38af5c0 Fixed, but leaving open so it doesn't fall off the fennec-1.0 triage list; I still want the blocking nom to be resolved.
(In reply to Kartikaya Gupta (:kats) from comment #12) > Fixed, but leaving open so it doesn't fall off the fennec-1.0 triage list; I > still want the blocking nom to be resolved. Actually, I'll just request aurora on the patch, seems simpler.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 620314 [details] [diff] [review] Save the pending displacement if we can't send it right away [Approval Request Comment] Regression caused by (bug #): User impact if declined: scrolling on pages with frames is barely possible Testing completed (on m-c, etc.): locally, on m-c for a couple of days now and i haven't heard any complaints Risk to taking this patch (and alternatives if risky): mobile-only, low risk String changes made by this patch: none
Attachment #620314 - Flags: approval-mozilla-aurora?
Kats, I've noticed with the iframes is that even while scrolling down it is very easy to cause unintentional horizontal scrolling. This creates jerkiness. This is a problem when scrolling within the iframe and just scrolling the entire page.
(In reply to Demetrius Schwab from comment #15) > Kats, I've noticed with the iframes is that even while scrolling down it is > very easy to cause unintentional horizontal scrolling. This creates > jerkiness. This is a problem when scrolling within the iframe and just > scrolling the entire page. Yes, the patches on bug 732364 should fix this behaviour by applying axis-locking to the scrolling in iframes. Do you still see the issue on the latest nightly?
No the issue appears to have been solved
Attachment #620314 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
This issue is not reproducible anymore on page with iframes. Closing bug as verified fixed on: Firefox 15.0a1 (2012-05-16) Firefox 14.0b3 Device: Galaxy Nexus OS: Android 4.0.2
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.