1803435 - Fire scrollend events to content by default

Status: RESOLVED FIXED

(Core :: Panning and Zooming, task, P2)

Description

[Dan Robertson (:dlrobertson)]

Set apz.scrollend-event.content.enabled=true to fire scrollend events to content by default, and let the feature ride the trains.

Comment 1

[Dan Robertson (:dlrobertson)]

Attachment: Bug 1803435 - Enable scrollend for content by default. r=hiro,botond

Enable firing the scrollend event to content by default.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b8a8ca798d9f
Enable scrollend for content by default. r=hiro,botond,emilio

Comment 3

[Cristian Tuns]

Backed out for causing wpt failures on math-global-event-handlers.tentative.html

• Backout link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-PASS | /mathml/relations/html5-tree/math-global-event-handlers.tentative.html | onscrollend: must be on the appropriate locations for GlobalEventHandlers - expected FAIL

Comment 4

[Dan Robertson (:dlrobertson)]

(In reply to Cristian Tuns from comment #3)

Backed out for causing wpt failures on math-global-event-handlers.tentative.html

• Backout link
• Push with failures
• Failure Log
• Failure line: TEST-UNEXPECTED-PASS | /mathml/relations/html5-tree/math-global-event-handlers.tentative.html | onscrollend: must be on the appropriate locations for GlobalEventHandlers - expected FAIL

Looks like new wpt tests that should pass with apz.scrollend-event.content.enabled=true. Updated the test expectations.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/848de6ab7ca9
Enable scrollend for content by default. r=hiro,botond,emilio

Comment 6

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/848de6ab7ca9

Comment 7

[Brian Thomas Smith [bsmth]]

We're tracking documentation changes for this in the following GitHub issue -> https://github.com/mdn/content/issues/22814

Comment 8

[Dan Robertson (:dlrobertson)]

Release Note Request (optional, but appreciated)
[Why is this notable]: Adds a new event scrollend which is fired when a scroll has completed.
[Affects Firefox for Android]:
[Suggested wording]: scrollend events fired to content by default on scroll completed.
[Links (documentation, blog post, etc)]:

• MDN Document: scrollend
• MDN Element: scrollend