1546057 - Don't snap back if scrollTo (or APZC) tries to scroll to positions where the positions are out of scrollable area and if snapping target elements covers the snapport

Status: RESOLVED FIXED

(Core :: Layout: Scrolling and Overflow, defect, P2)

Description

[Hiroyuki Ikezoe (:hiro)]

Attachment: scroll-snap-type.html

Steps to reproduce

1. Open the attaching file
2. Open the devtools console
3. Run scroller.scrollTo(0, 1400)

1400px is actually out of the boundary of the scroller element, the bottom edge is actually 1300px, but you will see the scroller element snaps to 1000px instead.

This is also one of the causes of the example in bug 1531589 comment 2.

Interestingly, Chrome has the same issue. :)

Comment 1

[Hiroyuki Ikezoe (:hiro)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=951a0dc97c8021f7c099543b783b79464e4c8f2c

Note that I guess this will also fix the weird bound back effect reported in bug 1545316 comment 0.

Comment 2

[Hiroyuki Ikezoe (:hiro)]

Attachment: Bug 1546057 - Use the clamped destination position for overflow checks. r?botond

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/eda47278259f
Use the clamped destination position for overflow checks. r=botond

Comment 4

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/eda47278259f

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16538 for changes under testing/web-platform/tests