Closed Bug 1539687 Opened 1 year ago Closed 1 year ago

Visual smooth scroll is incorrectly clamped to the layout scroll range

Categories

(Core :: Panning and Zooming, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: botond, Assigned: botond)

References

Details

Attachments

(2 files)

While testing a patch for bug 1531531, which uses the visual smooth scrolling API added in bug 1531535, Randall discovered that the API is still clamping the visual scroll offset to the layout scroll range.

The problem here is clamping mDestination. I thought that leave mApzSmoothScrollDestination unclamped would be sufficient, but it's not, we actually pick up mDestination as the smooth scroll destination.

Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/152457c26513
Do not limit visual smooth scrolling to the layout scroll range. r=kats
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
Pushed by bballo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b6231c6dcd00
Error out if a property name is misspelt in a subtest entry. r=kats
You need to log in before you can comment on or make changes to this bug.