Closed Bug 941328 Opened 6 years ago Closed 6 years ago

Double-tap to zoom out always scrolls to the top of the page

Categories

(Firefox for Metro Graveyard :: Install/Update, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 28

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

Attachments

(1 file)

Instead of scrolling to the top of the page, we should zoom out to an area of the page that is roughly centered on the tapped/zoomed location, so that the same part of the document is still visible.
Component: Input → Install/Update
OS: Windows 8.1 → All
Hardware: x86_64 → All
Summary: Double-tap to zoom out always scrolls to the top of the page → Defect - Double-tap to zoom out always scrolls to the top of the page
Whiteboard: [triage] feature=defect c=tbd u=tbd p=0
Attached patch patchSplinter Review
It turns out APZC will automatically zoom out if we pass an empty rect.
Assignee: nobody → mbrubeck
Status: NEW → ASSIGNED
Attachment #8335665 - Flags: review?(ally)
Hey Matt, can you provide a point value.
Flags: needinfo?(mbrubeck)
p=1 (or just add this a work item to the double-tap story)
Flags: needinfo?(mbrubeck)
No longer blocks: metrov1backlog
Summary: Defect - Double-tap to zoom out always scrolls to the top of the page → Double-tap to zoom out always scrolls to the top of the page
Whiteboard: [triage] feature=defect c=tbd u=tbd p=0
Comment on attachment 8335665 [details] [diff] [review]
patch

Review of attachment 8335665 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. I wonder why (0,0,0,0) would no-op for me.
Attachment #8335665 - Flags: review?(ally) → review+
https://hg.mozilla.org/mozilla-central/rev/69fb4a0fc17f
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 28
You need to log in before you can comment on or make changes to this bug.