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

Status: RESOLVED FIXED

(Core :: Panning and Zooming, defect)

Description

[Matt Brubeck (:mbrubeck)]

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.

Comment 1

[Marco Mucci [:MarcoM]]

Hey Matt, can you provide a point value.

Comment 2

[Matt Brubeck (:mbrubeck)]

Attachment: 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.

Comment 3

[Matt Brubeck (:mbrubeck)]

(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.

Comment 4

[Matt Brubeck (:mbrubeck)]

https://tbpl.mozilla.org/?tree=Try&rev=ced11aa2eec1

Comment 5

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/73aee4abf07b

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/73aee4abf07b