Closed Bug 1266154 Opened 3 years ago Closed 3 years ago
Momentum scrolling after scrolling a scroll frame with scroll snapping
MozReview Request: Bug 1266154 - Don't start a scroll snap animation if we're already at the destination. r=kats
58 bytes, text/x-review-board-request
I'm splitting out this bug from bug 1265513, which concerns the interaction between scroll snapping and handoff during panning (as opposed to after the finger is lifted). There are two issues here, summarized in bug 1265513 comment 6. The first one I have a fix for (originally posted in bug 1265513 comment 5). The second one I argue is expected (in bug 1265513 comment 6), but I'm open to being convinced otherwise.
Assignee: nobody → botond
See Also: → 1265513
Review commit: https://reviewboard.mozilla.org/r/47739/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/47739/
Attachment #8743401 - Flags: review?(bugmail.mozilla)
Comment on attachment 8743401 [details] MozReview Request: Bug 1266154 - Don't start a scroll snap animation if we're already at the destination. r=kats Carrying r=kats from bug 1265513 comment 7.
Attachment #8743401 - Flags: review?(bugmail.mozilla) → review+
Try push that includes this patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f53dd840d47c
Landed (with bug 1265515 as a ride-along).
You need to log in before you can comment on or make changes to this bug.