Closed
Bug 704690
Opened 13 years ago
Closed 13 years ago
html anchors do not pan to the right spot
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: nhirata, Assigned: kats)
References
()
Details
Attachments
(2 files, 2 obsolete files)
4.12 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
4.74 KB,
patch
|
kats
:
review+
|
Details | Diff | Splinter Review |
1. go to http://people.mozilla.com/~nhirata/html_tp/jumptag.html 2. click on the link "Go to Bottom" Expected: Goes to the bottom of the page Actual: in some cases it doesn't move. HTC Flyer; 20111122; Android 2.3
Comment 1•13 years ago
|
||
related to Bug 703620 ?
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → kgupta
Assignee | ||
Comment 2•13 years ago
|
||
Attachment #578056 -
Flags: review?(chrislord.net)
Assignee | ||
Comment 3•13 years ago
|
||
In the test URL on this bug, I often end up flinging/snapping the page when clicking on the "bottom" link to go back to the top. This was causing the viewport to get clobbered by the FlingRunnable, so I needed this patch to make it work.
Attachment #578057 -
Flags: review?(chrislord.net)
Comment 4•13 years ago
|
||
Comment on attachment 578056 [details] [diff] [review] (1/2) Force viewport update on anchor jump Review of attachment 578056 [details] [diff] [review]: ----------------------------------------------------------------- r+, but would prefer to see the comment below addressed. ::: mobile/android/chrome/content/browser.js @@ +1185,5 @@ > + gecko: { > + type: "Viewport:Update", > + viewport: JSON.stringify(this.viewport) > + } > + }); Why doesn't this use sendViewportUpdate? I would suggest refactoring sendViewportUpdate so it can be used for things like this, so we don't have these last few lines in multiple places.
Attachment #578056 -
Flags: review?(chrislord.net) → review+
Comment 5•13 years ago
|
||
Comment on attachment 578057 [details] [diff] [review] (2/2) Abort flings on viewport update Review of attachment 578057 [details] [diff] [review]: ----------------------------------------------------------------- r+, with the comment addressed (either via a rationalisation or a code change, either is fine by me) ::: mobile/android/base/ui/PanZoomController.java @@ +133,5 @@ > public void geometryChanged() { > populatePositionAndLength(); > } > > + public void viewportMetricsChanged() { Can we just have a boolean flag in geometryChanged, something like aCancelTouch or aReset (or some other better name that escapes me)? Having a separate function for what is essentially the same event, but from different sources could become confusing down the line.
Attachment #578057 -
Flags: review?(chrislord.net) → review+
Assignee | ||
Comment 6•13 years ago
|
||
Updated with suggested refactoring.
Attachment #578056 -
Attachment is obsolete: true
Attachment #578243 -
Flags: review+
Assignee | ||
Comment 7•13 years ago
|
||
(In reply to Chris Lord [:cwiiis] from comment #5)> > Can we just have a boolean flag in geometryChanged, something like > aCancelTouch or aReset (or some other better name that escapes me)? Having a > separate function for what is essentially the same event, but from different > sources could become confusing down the line. Yeah, that makes more sense, fixed.
Attachment #578057 -
Attachment is obsolete: true
Attachment #578244 -
Flags: review+
Assignee | ||
Comment 8•13 years ago
|
||
https://hg.mozilla.org/projects/birch/rev/ec36c2b415cc https://hg.mozilla.org/projects/birch/rev/cba7acb26616
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 10•13 years ago
|
||
Samsung Galaxy SII (Android 2.3.4) 20111202040206 http://hg.mozilla.org/projects/birch/rev/e2a54aafac18
Status: RESOLVED → VERIFIED
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
•