Skip to content

RN: Re-enable useInsertionEffectsForAnimations#48708

Closed
yungsters wants to merge 1 commit intofacebook:mainfrom
yungsters:export-D68217144
Closed

RN: Re-enable useInsertionEffectsForAnimations#48708
yungsters wants to merge 1 commit intofacebook:mainfrom
yungsters:export-D68217144

Conversation

@yungsters
Copy link
Contributor

Summary:
{D68171721} facebook/react-native#48678 mitigated the bug that necessitated this revert.

Changelog:
[General][Changed] - (Reapply) The AnimatedNode graph will not occur during the insertion effect phase, which means animations can now be reliably started during layout effects.

Differential Revision: D68217144

Summary:
{D68171721} [facebook#48678](facebook#48678) mitigated the bug that necessitated this revert.

Changelog:
[General][Changed] - (Reapply) The `AnimatedNode` graph will not occur during the insertion effect phase, which means animations can now be reliably started during layout effects.

Differential Revision: D68217144
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner labels Jan 15, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D68217144

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e0c0476.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @yungsters in e0c0476

When will my fix make it into a release? | How to file a pick request?

@yungsters yungsters deleted the export-D68217144 branch March 13, 2025 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants