Skip to content

Cancel animation when a custom Timeline is used#35567

Merged
sebmarkbage merged 1 commit intofacebook:mainfrom
sebmarkbage:gesturefirefoxbug
Jan 20, 2026
Merged

Cancel animation when a custom Timeline is used#35567
sebmarkbage merged 1 commit intofacebook:mainfrom
sebmarkbage:gesturefirefoxbug

Conversation

@sebmarkbage
Copy link
Copy Markdown
Contributor

Follow up to #35559.

The clean up function of the custom timeline doesn't necessarily clean up the animation. Just the timeline's internal state.

This affects Firefox which doesn't support ScrollTimeline so uses the polyfill's custom timeline.

The clean up function of the custom timeline doesn't necessarily clean up
the animation. Just the timeline's internal state.
@sebmarkbage sebmarkbage requested a review from acdlite January 20, 2026 01:52
@meta-cla meta-cla Bot added the CLA Signed label Jan 20, 2026
@sebmarkbage sebmarkbage merged commit d290875 into facebook:main Jan 20, 2026
14 of 15 checks passed
@github-actions github-actions Bot added the React Core Team Opened by a member of the React Core Team label Jan 20, 2026
@react-sizebot
Copy link
Copy Markdown

Comparing: d343c39...817bd79

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB +0.11% 1.88 kB 1.88 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 608.73 kB 608.73 kB = 107.65 kB 107.65 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB +0.05% 1.88 kB 1.88 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 674.61 kB 674.66 kB = 118.59 kB 118.59 kB
facebook-www/ReactDOM-prod.classic.js = 694.11 kB 694.11 kB = 122.02 kB 122.02 kB
facebook-www/ReactDOM-prod.modern.js = 684.50 kB 684.50 kB = 120.42 kB 120.42 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 817bd79

github-actions Bot pushed a commit that referenced this pull request Jan 20, 2026
Follow up to #35559.

The clean up function of the custom timeline doesn't necessarily clean
up the animation. Just the timeline's internal state.

This affects Firefox which doesn't support ScrollTimeline so uses the
polyfill's custom timeline.

DiffTrain build for [d290875](d290875)
github-actions Bot pushed a commit to shlomiNugarker/react that referenced this pull request Jan 20, 2026
Follow up to facebook#35559.

The clean up function of the custom timeline doesn't necessarily clean
up the animation. Just the timeline's internal state.

This affects Firefox which doesn't support ScrollTimeline so uses the
polyfill's custom timeline.

DiffTrain build for [d290875](facebook@d290875)
github-actions Bot pushed a commit to shlomiNugarker/react that referenced this pull request Jan 20, 2026
Follow up to facebook#35559.

The clean up function of the custom timeline doesn't necessarily clean
up the animation. Just the timeline's internal state.

This affects Firefox which doesn't support ScrollTimeline so uses the
polyfill's custom timeline.

DiffTrain build for [d290875](facebook@d290875)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed React Core Team Opened by a member of the React Core Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants