ZoomToRect (double-tap) causes the rect to swoop around instead of zooming directly to it

RESOLVED FIXED in mozilla28

Status

()

Core
Panning and Zooming
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Tracking

28 Branch
mozilla28
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
When you double-tap to zoom to an element, rather than scaling and scrolling the element linearly into place, the element seems to scroll first down and to the right, then up and to the left, taking a "swoopy" path around the screen before finally ending up centered on screen.

Updated

4 years ago
Blocks: 838081
Summary: ZoomToRect (double-tap) causes the rect to swoop around instead of zooming directly to it → Defect - ZoomToRect (double-tap) causes the rect to swoop around instead of zooming directly to it
Whiteboard: [triage] feature=defect c=tbd u=tbd p=0
(Assignee)

Updated

4 years ago
Assignee: nobody → mbrubeck
Blocks: 886321
Status: NEW → ASSIGNED
Hey Matt, can you provide a point value.
Flags: needinfo?(mbrubeck)
(Assignee)

Comment 2

4 years ago
Created attachment 8335711 [details] [diff] [review]
patch

This happened because the scale changes and the scroll changes were moving at different rates.  This adjusts the scale animation to correctly stay "in sync" with the scroll animation.
Attachment #8335711 - Flags: review?(botond)
(Assignee)

Comment 3

4 years ago
(In reply to Marco Mucci [:MarcoM] from comment #1)
> Hey Matt, can you provide a point value.

I'd consider this a work item for the double-tap story (bug 831986) or the APZC story (bug metro-apzc), but if we want to treat it as a defect then p=1.
Flags: needinfo?(mbrubeck)

Updated

4 years ago
No longer blocks: 838081
Summary: Defect - ZoomToRect (double-tap) causes the rect to swoop around instead of zooming directly to it → ZoomToRect (double-tap) causes the rect to swoop around instead of zooming directly to it
Whiteboard: [triage] feature=defect c=tbd u=tbd p=0

Updated

4 years ago
Attachment #8335711 - Flags: review?(botond) → review+
https://hg.mozilla.org/mozilla-central/rev/73aee4abf07b
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
You need to log in before you can comment on or make changes to this bug.