Closed Bug 595229 Opened 13 years ago Closed 13 years ago

Page pans at start of multitouch zoom gesture

Categories

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

Product:
Firefox for Android Graveyard
Component:
Panning/Zooming
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: mbrubeck, Assigned: mbrubeck)

Details

Attachments

(1 file)

patch
2.90 KB, patch
stechz
: review+
Details | Diff | Splinter Review 
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 patch — — Splinter 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: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.