Skip to content
This repository was archived by the owner on Jan 10, 2020. It is now read-only.

Update AfterEffects and iOS Keyframes to Support XY Rotation #110

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jacksontaylor13
Copy link

After Effects Script: Added in functionality to support the XY rotation on a 3D layer in After Effects. If you perform the transformation on the Transform of the Shape Layer you will be able to animate the 3-D axes also. Currently the support is only for Z-axis rotation, so this PR will add in the new functionality. In the Keyframes file, once exported, the keys used for X and Y rotation are X_ROTATION and Y_ROTATION, respectively.

iOS SDK: In order to interpret and handle the new 3D layer transformations, the iOS SDK will now be able to intepret the values and insert the animations accordingly. Using a new KFAxis enumeration on the KFVectorAnimationLayer class, you are able to specify which axis the rotation animation should occur on. This will hopefully help abstract some of the code in that file, to prevent duplication.

After Effects Script: Added in functionality to support the XY rotation on a 3D layer in After Effects. If you perform the transformation on the  of the  you will be able to animate the 3-D axes also. Currently the support is only for Z-axis rotation, so this PR will add in the new functionality. In the Keyframes file, once exported, the keys used for X and Y rotation are  and , respectively.

iOS SDK: In order to interpret and handle the new 3D layer transformations, the iOS SDK will now be able to intepret the values and insert the animations accordingly. Using a  new  enumeration on the  class, you are able to specify which axis the rotation animation should occur on. This will hopefully help abstract some of the code in that file, to prevent duplication.
@jacksontaylor13
Copy link
Author

Since this PR deals with rotation, also included a fix for issue #106.

@facebook-github-bot
Copy link
Contributor

@jacksontaylor13 updated the pull request - view changes

Copy link
Contributor

@renyu-io renyu-io left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your changes 👍

@facebook-github-bot
Copy link
Contributor

@LazyChild has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@jacksontaylor13 updated the pull request - view changes - changes since last import

@facebook-github-bot
Copy link
Contributor

@LazyChild has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@kjhgg
Copy link

kjhgg commented Apr 30, 2018

Hf

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants