Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Opacity not working #3

Open
felippenardi opened this issue Jul 21, 2018 · 8 comments
Open

Opacity not working #3

felippenardi opened this issue Jul 21, 2018 · 8 comments

Comments

@felippenardi
Copy link

I'm trying to target Opacity and it won't work. Neither does with Shape Layers and Null Objects.

Can you try targeting the Opacity and confirm it is not working?

@bodymovin
Copy link
Owner

Can you share your implementation?

@felippenardi
Copy link
Author

@bodymovin Thanks for replying so fast!

This is the implementation:

Starting the animation using react-lottie: https://github.com/felippenardi/lottie-face-control/blob/master/src/App.js#L27

Using lottie-api to modify properties: https://github.com/felippenardi/lottie-face-control/blob/master/src/utils/LottieWithAnimationControl.js#L11

Note that the controlling the position of JoyStkCtrl01 works, and modifying the scale of the Cursor (not in the code) does also works. But when it comes to Opacity, the addValueCallback is never triggered.

Here is my after effects:
screen shot 2018-07-21 at 16 29 47

Here is a live preview of the above repo:
https://lottie-face-control-okfpwprnjr.now.sh/

@bodymovin
Copy link
Owner

Hi, it should be fixed now.

@felippenardi
Copy link
Author

You are the best!

@felippenardi
Copy link
Author

@bodymovin I tried to fix it myself, but I was replacing po for ks, which was the property my Json was holding the opacity. How did you get to o?

@bodymovin
Copy link
Owner

I also develop the player, so I knew the name of the property :)

@felippenardi
Copy link
Author

Just curious on why opacity was named "ks" inside my json

@bodymovin
Copy link
Owner

it shouldn't.
ks is a group of properties including opacity and transforming properties

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants