Polyfill onScrollEnd Event in Safari #32427
Open
+281
−4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
React Core Team
|
Comparing: 3607f48...399aa22 Critical size changesIncludes critical production bundles, as well as any change greater than 2%:
Significant size changesIncludes any change greater than 0.2%: Expand to show |
|
Note that at one point in the past the |
Enabled in native and test-renderer since they shouldn't affect them anyway. Makes it easier to see that it's safe to ship.
sebmarkbage
force-pushed
the
scrollend
branch
from
d255e40 to
dead6ae
Compare
|
There's some movement in Safari. It's part of the roadmap for 2025. https://webkit.org/blog/16458/announcing-interop-2025/#scrollend-event But we still need to polyfill for now and for older versions. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We added support for
onScrollEndin #26789 but it only works in Chrome and Firefox. Safari still doesn't supportscrollendand there's no indication that they will anytime soon so this polyfills it.While I don't particularly love our synthetic event system this tries to stay within the realm of how our other polyfills work. This implements all
onScrollEndevents as a plugin.The basic principle is to first feature detect the
onscrollendDOM property to see if there's native support and otherwise just use the native event.Then we listen to
scrollevents and set a timeout. If we don't get any more scroll events before the timeout we fireonScrollEnd. Basically debouncing it. If we're currently pressing down on touch or a mouse then we wait until it is lifted such as if you're scrolling with a finger or using the scrollbars on desktop but isn't currently moving.If we do get any native events even though we're in polyfilling mode, we use that as an indication to fire the
onScrollEndearly.Part of the motivation is that this becomes extra useful pair for #32422. We also probably need these events to coincide with other gesture related internals so you're better off using our polyfill so they're synced.