Closed
Bug 1091128
Opened 10 years ago
Closed 10 years ago
Max velocity cap only applies when scrolling downwards/rightwards
Categories
(Core :: Panning and Zooming, defect)
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
1.36 KB,
patch
|
botond
:
review+
|
Details | Diff | Splinter Review |
The velocity cap added in bug 976035 only caps the upper bound of velocities, not the lower bound. So when scrolling upwards/leftwards there is effectively no cap.
Assignee | ||
Updated•10 years ago
|
status-b2g-v1.4:
--- → affected
status-b2g-v2.0:
--- → affected
status-b2g-v2.1:
--- → affected
status-b2g-v2.2:
--- → affected
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8513717 -
Flags: review?(botond)
Comment 2•10 years ago
|
||
Comment on attachment 8513717 [details] [diff] [review] Take sign into consideration Review of attachment 8513717 [details] [diff] [review]: ----------------------------------------------------------------- Good catch! ::: gfx/layers/apz/src/Axis.cpp @@ +48,5 @@ > } > > float newVelocity = mAxisLocked ? 0.0f : (float)(mPos - aPos) / (float)(aTimestampMs - mPosTimeMs); > if (gfxPrefs::APZMaxVelocity() > 0.0f) { > + bool negative = (newVelocity < 0); Let's call this 'velocityIsNegative'.
Attachment #8513717 -
Flags: review?(botond) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Updated variable name and landed: https://hg.mozilla.org/integration/b2g-inbound/rev/08655880fafc
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/08655880fafc
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•