Animation Backend init#53526
Conversation
|
@bartlomiejbloniarz has imported this pull request. If you are a Meta employee, you can view this in D81309558. |
|
This pull request was exported from Phabricator. Differential Revision: D81309558 |
Summary: ## Changelog: <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [ANDROID|GENERAL|IOS|INTERNAL] [BREAKING|ADDED|CHANGED|DEPRECATED|REMOVED|FIXED|SECURITY] - Message For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests Pull Request resolved: #53526 Differential Revision: D81309558 Pulled By: bartlomiejbloniarz
17e676b to
5573403
Compare
This comment was marked as spam.
This comment was marked as spam.
Job Summary for GradleTest All :: test_android_helloworld
|
Job Summary for GradleTest All :: test_android_helloworld
|
Job Summary for GradleTest All :: run_fantom_tests
|
Job Summary for GradleTest All :: run_fantom_tests
|
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has imported this pull request. If you are a Meta employee, you can view this in D81309558. |
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
Summary: This diff adds AnimatedProps class that allows prop diffs to be passed to the backend. Currently it is implemented through subclassing, but the implementation details are subject to change. To simplify integration with animation frameworks AnimatedPropsBuilder class was created - frameworks can use it to prepare their diffs. ## Changelog: <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [GENERAL] [ADDED] - AnimatedProps.h AnimatedPropsBuilder.h [GENERAL] [CHANGED] - AnimationBackend initialisation and style updates For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests Pull Request resolved: #53526 Test Plan: js1 fantom AnimatedBackend-itest.js js1 fantom Animated-itest.js Differential Revision: D81309558 Pulled By: bartlomiejbloniarz
8ed3a10 to
e9fe8cf
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
e9fe8cf to
dcc50a1
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
dcc50a1 to
f721579
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
f721579 to
9a35201
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
Summary: This diff adds AnimatedProps class that allows prop diffs to be passed to the backend. Currently it is implemented through subclassing, but the implementation details are subject to change. To simplify integration with animation frameworks AnimatedPropsBuilder class was created - frameworks can use it to prepare their diffs. ## Changelog: <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [GENERAL] [ADDED] - AnimatedProps.h AnimatedPropsBuilder.h [GENERAL] [CHANGED] - AnimationBackend initialisation and style updates For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests Pull Request resolved: #53526 Test Plan: js1 fantom AnimatedBackend-itest.js js1 fantom Animated-itest.js Differential Revision: D81309558 Pulled By: bartlomiejbloniarz
9a35201 to
635f89a
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
635f89a to
ce80217
Compare
Job Summary for GradleTest All :: run_fantom_tests
|
Job Summary for GradleTest All :: run_fantom_tests
|
|
@bartlomiejbloniarz has exported this pull request. If you are a Meta employee, you can view the originating diff in D81309558. |
Summary: This diff adds AnimatedProps class that allows prop diffs to be passed to the backend. Currently it is implemented through subclassing, but the implementation details are subject to change. To simplify integration with animation frameworks AnimatedPropsBuilder class was created - frameworks can use it to prepare their diffs. ## Changelog: <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [GENERAL] [ADDED] - AnimatedProps.h AnimatedPropsBuilder.h [GENERAL] [CHANGED] - AnimationBackend initialisation and style updates For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests Pull Request resolved: #53526 Test Plan: js1 fantom AnimatedBackend-itest.js js1 fantom Animated-itest.js Reviewed By: zeyap Differential Revision: D81309558 Pulled By: bartlomiejbloniarz
baddce7 to
942f001
Compare
Reviewed By: zeyap, sammy-SC Differential Revision: D80809364
Differential Revision: D81138133
Reviewed By: zeyap Differential Revision: D81137954
Summary: This diff adds AnimatedProps class that allows prop diffs to be passed to the backend. Currently it is implemented through subclassing, but the implementation details are subject to change. To simplify integration with animation frameworks AnimatedPropsBuilder class was created - frameworks can use it to prepare their diffs. ## Changelog: <!-- Help reviewers and the release process by writing your own changelog entry. Pick one each for the category and type tags: [GENERAL] [ADDED] - AnimatedProps.h AnimatedPropsBuilder.h [GENERAL] [CHANGED] - AnimationBackend initialisation and style updates For more details, see: https://reactnative.dev/contributing/changelogs-in-pull-requests Pull Request resolved: #53526 Test Plan: js1 fantom AnimatedBackend-itest.js js1 fantom Animated-itest.js Reviewed By: zeyap Differential Revision: D81309558 Pulled By: bartlomiejbloniarz
942f001 to
16989eb
Compare
|
@bartlomiejbloniarz merged this pull request in e091759. |
Summary:
Initialize the Animation Backend - a unified way to modify view styles, that can be used by animation libraries. In it's current design it allows libraries to register a callback on each frame, that pushes a list of mutations to the platform. The idea is to pass the props as diffs to minimize the cost. The current implementation of
AnimatedPropsis subject to change, as it is not really optimised and doesn't cover many props (though there is afolly::dynamicfallback) - this is just a starting point. Animation libraries should rely onAnimatedPropsBuilder, since it should remain fairly stable over time.Changelog:
Test Plan:
Run the fantom test for the Backend.