Closed Bug 598368 Opened 9 years ago Closed 9 years ago

Double tap zoom does not clamp scale until animation finishes

Categories

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

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

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

(
URL
)

Details

(Keywords: testcase)

Attachments

(2 files)

clamp the zoom level before animating
836 bytes, patch
mfinkle
: review+
Details | Diff | Splinter Review
test case
179 bytes, text/html
Details 
On pages that set a maximum-scale, double-tap can animate to a disallowed zoom level, and then jump back to the allowed range when the animation finishes.  This happens on touch.facebook.com and in the attached test case.
Attachment #477181 - Flags: review?(mark.finkle)
Attached file test case — 
Comment on attachment 477181 [details] [diff] [review]
clamp the zoom level before animating

anyway to add this to the viewport tests?
Attachment #477181 - Flags: review?(mark.finkle) → review+
Whiteboard: [fennec-checkin-postb1]
pushed
http://hg.mozilla.org/mobile-browser/rev/f86a3e00b641
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [fennec-checkin-postb1]
verified:
Mozilla/5.0 (Maemo; Linux armv71; rv:2.0b7pre) Gecko/201001005 Firefox/4.0b7pre Fennec/4.0b1
Mozille/5.0 (Android; Linux armv71; rv:2.0b7pre) Gecko20101005 Firefox/4.0b7pre Fennec/4.0b1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.