Closed Bug 1543224 Opened 5 years ago Closed 5 years ago

Don't accelerate zero-velocity flings into non-zero velocity flings

Categories

(Core :: Panning and Zooming, defect, P3)

defect

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

Spinoff from bug 1541644. While scrolling around on the wikipedia pastries table, I noticed that sometimes a vertical scroll attempt would result in horizontal scrolling instead. The lack of vertical scrolling is explained by bug 1541644 comment 6. And the seemingly-random horizontal scrolling was actually a result of fling acceleration. This happened because the previous scroll I did had produced some horizontal fling velocity, and then the new scroll, even though it had a zero horizontal scroll component, got accelerated into a non-zero horizontal fling.

I think it seems weird to accelerate flings with a zero horizontal velocity component into a nonzero horizontal velocity component, so I wrote a patch to eliminate that.

Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1ef7a84a9533
Don't fling-accelerate a zero velocity to a nonzero velocity. r=botond
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: