Closed Bug 653772 Opened 14 years ago Closed 14 years ago

Add a minimum velocity for kinetic panning

Categories

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

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 6

People

(Reporter: mbrubeck, Assigned: mbrubeck)

References

Details

(Keywords: polish)

Attachments

(1 file)

Attached patch patchSplinter Review
Since bug 639179 landed, it's a little *too* easy to start very slow kinetic with very little movement. Among other things, this makes it hard to hide the sidebars without scrolling the page (when the page is zoomed in). This patch improves things for me by setting a minimum velocity for kinetic panning to start. The minimum of 400 pixels/s is based on manual tuning.
Attachment #529136 - Flags: review?(ben)
Attachment #529136 - Flags: review?(ben) → review+
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Whiteboard: [has patch]
Target Milestone: --- → Firefox 6
Is there a way for us to verify this?
If you remove your finger from the screen while scrolling *very* slowly (around 2 cm/second), scrolling should now stop immediately, rather than continuing to pan with momentum as it would before this patch landed.
Thanks Matt, verified fixed on build: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110505 Firefox/6.0a1 Fennec/6.0a1 Device: LG Optimus 2X
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: