-
Notifications
You must be signed in to change notification settings - Fork 10
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
Animate AI Assistant button and panel #1097
Conversation
Test Results 1 files ±0 1 suites ±0 8m 4s ⏱️ +24s For more details on these errors, see this check. Results for commit df61a89. ± Comparison against base commit f76bbd3. ♻️ This comment has been updated with latest results. |
f836a10
to
cd5b687
Compare
I remember that this was still something we have to finalise after finishing the implementation of the scale correction/transform based animating. I don't think the current version has a edit: to add to that, I think with the current timeline functionality (animating every sprite for the entire duration instead of just when it's animated), anchoring, padding etc. we may be able to get rid of lockStyles by integrating it into the keyframes. |
Thanks @nickschot. I updated this PR to use a StaticBehavior to fix the width during the animation. I generalized the fill behavior so that you can opt StaticBehavior into forward filling like the other behaviors (except Wait) already did by default. Integrating the lockStyles into the timeline is an interesting idea that likely simplifies the overall code and mental model. |
… Assistant Panel animation
… accept splattributes
…aticBehavior treatment
… forward fill - when using StaticBehavior with SpringBehavior, you don't know how long the animation will be, and you may want a static animation to persist for the duration of the animation
After discussion with the team and with Chris, we decided to defer animation work on host, so I'll close this PR after extracting the non-animation changes from it into new PRs |
Screen.Recording.2024-03-20.at.4.03.35.PM.mov