Skip to content

Wire up startViewTransitionReadyFinished in Fabric#36246

Open
zeyap wants to merge 1 commit intofacebook:mainfrom
zeyap:fabric-view-transition-ready-finished
Open

Wire up startViewTransitionReadyFinished in Fabric#36246
zeyap wants to merge 1 commit intofacebook:mainfrom
zeyap:fabric-view-transition-ready-finished

Conversation

@zeyap
Copy link
Copy Markdown
Contributor

@zeyap zeyap commented Apr 9, 2026

Summary

  • Imports startViewTransitionReadyFinished from nativeFabricUIManager in ReactFiberConfigFabricWithViewTransition
  • Calls fabricStartViewTransitionReadyFinished() when the view transition ready promise resolves

This is not a config function, but it's helpful to have it notify fabric ViewTransition runtime when ready callback is done. Right now we're testing animation kicked off from view transition event handlers, this is signal to know when animations that belong to a transition have all started.

Test plan

  • Existing Fabric renderer tests should continue to pass
  • View transition ready callback now notifies the native module when finished

Import startViewTransitionReadyFinished from nativeFabricUIManager and
call it when the view transition ready promise resolves.
@meta-cla meta-cla bot added the CLA Signed label Apr 9, 2026
@react-sizebot
Copy link
Copy Markdown

Comparing: 705268d...407d1a9

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 = 1.88 kB 1.88 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 612.91 kB 612.91 kB = 108.30 kB 108.30 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.84 kB 6.84 kB = 1.88 kB 1.88 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 678.85 kB 678.85 kB = 119.27 kB 119.27 kB
facebook-www/ReactDOM-prod.classic.js = 698.24 kB 698.24 kB = 122.65 kB 122.65 kB
facebook-www/ReactDOM-prod.modern.js = 688.55 kB 688.55 kB = 121.03 kB 121.03 kB
facebook-www/ESLintPluginReactHooks-prod.classic.js New file 0.00 kB 2,038.50 kB New file 0.00 kB 297.56 kB
facebook-www/ESLintPluginReactHooks-prod.modern.js New file 0.00 kB 2,038.50 kB New file 0.00 kB 297.56 kB

Significant size changes

Includes any change greater than 0.2%:

Expand to show
Name +/- Base Current +/- gzip Base gzip Current gzip
facebook-www/ESLintPluginReactHooks-prod.classic.js New file 0.00 kB 2,038.50 kB New file 0.00 kB 297.56 kB
facebook-www/ESLintPluginReactHooks-prod.modern.js New file 0.00 kB 2,038.50 kB New file 0.00 kB 297.56 kB

Generated by 🚫 dangerJS against 407d1a9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants