Closed Bug 595229 Opened 14 years ago Closed 14 years ago

Page pans at start of multitouch zoom gesture

Categories

(Firefox for Android Graveyard :: Panning/Zooming, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Details

Attachments

(1 file)

At the start of a pinch zoom, the page pans to center the point where the gesture started.  So e.g. if you touch near the bottom of the screen, the page will pan up so the region you touched is centered.
Attached patch patchSplinter Review
This is a patch against mobile2 with the patch from bug 595052 applied, but the same changes should work in mozilla-central too.
Attachment #474125 - Flags: review?(webapps)
Comment on attachment 474125 [details] [diff] [review]
patch

Patches look good to me.
Attachment #474125 - Flags: review?(webapps) → review+
Pushed to mobile2:
http://hg.mozilla.org/users/bstover_mozilla.com/mobile2/rev/a9fb288e37cb

I'm not going to bother backporting this to mobile-browser, since the multitouch code there is going away.  This can be verified in a cedar+mobile2 build, or after mobile2 is merged back to mobile-browser (hopefully this week).
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: