Skip to content

feat: Move Animated to @react-native/animated - #35467

Closed
gabrieldonadel wants to merge 1 commit into
react:mainfrom
gabrieldonadel:feat/add-animated-package
Closed

gabrieldonadel wants to merge 1 commit into
react:mainfrom
gabrieldonadel:feat/add-animated-package

Conversation

@gabrieldonadel

@gabrieldonadel gabrieldonadel commented Nov 24, 2022 •

Copy link
Copy Markdown
Collaborator

Summary

This PR moves the Animated module to a separate package called @react-native/animated located under packages/animated as proposed on #35263

Changelog

[General] [Changed] - Move Animated to @react-native/animated package

Test Plan

  1. Open the RNTester app and navigate to Animated page
  2. Test animated functionalities through the many sections
Screen.Recording.2022-11-25.at.01.46.36.mov

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 24, 2022
@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch from 7c0019d to e2d9c39 Compare November 25, 2022 03:57
@github-actions

github-actions Bot commented Nov 25, 2022 •

Copy link
Copy Markdown
Warnings
⚠️ 🔒 package.json - Changes were made to package.json. This will require a manual import by a Facebook employee.

Generated by 🚫 dangerJS against 20ea260

@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch 8 times, most recently from fea3770 to 9c3552f Compare November 25, 2022 05:03
@gabrieldonadel
gabrieldonadel marked this pull request as ready for review November 25, 2022 05:03
@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch 3 times, most recently from 190672f to a9a1df1 Compare November 30, 2022 12:26
@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch 2 times, most recently from 2c2393a to 58cb383 Compare December 4, 2022 17:46
@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch 2 times, most recently from ed749c3 to 0bcf394 Compare December 20, 2022 21:15
@gabrieldonadel
gabrieldonadel force-pushed the feat/add-animated-package branch from 0bcf394 to 20ea260 Compare December 20, 2022 21:21
@necolas

necolas commented Mar 28, 2023

Copy link
Copy Markdown

Looks like we need to update this because of the monorepo moves

@gabrieldonadel

Copy link
Copy Markdown
Collaborator Author

Looks like we need to update this because of the monorepo moves

To be honest there are so many conflicts with the new monorepo structure that I'll probably just start this from scratch and force push a new commit

@necolas

necolas commented Apr 11, 2023 •

Copy link
Copy Markdown

An issue that I recently encountered with the RN Animated is that React.useLayoutEffect in useAnimatedProps causes errors during SSR on Web. IIRC there is a proposal for React.useLayoutEffect to change to a noop during SSR, but if that doesn't land soon, then we'd need to use a user-space useLayoutEffect that avoids the issue.

necolas/react-native-web@338cce1

@facebook-github-bot facebook-github-bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Jul 20, 2023
@DylanTet

Copy link
Copy Markdown

Am i able to just fetch from this PR to set my branch from this?

@github-actions

Copy link
Copy Markdown

This PR is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale There has been a lack of activity on this issue and it may be closed soon. label Apr 29, 2024
@github-actions

github-actions Bot commented May 7, 2024

Copy link
Copy Markdown

This PR was closed because it has been stalled for 7 days with no activity.

@github-actions github-actions Bot closed this May 7, 2024
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. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. Stale There has been a lack of activity on this issue and it may be closed soon.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants