Skip to content

Add 3D rotation support for Lottie animations#2666

Merged
gpeal merged 3 commits into
airbnb:masterfrom
eval3:3d-layers
Oct 23, 2025
Merged

Add 3D rotation support for Lottie animations#2666
gpeal merged 3 commits into
airbnb:masterfrom
eval3:3d-layers

Conversation

@eval3

@eval3 eval3 commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

The purpose is to solve the problem of Android Lottie not supporting 3D layers.

@eval3

eval3 commented Oct 9, 2025

Copy link
Copy Markdown
Contributor Author

Please, approval workflow and allow me to push to master branch

@github-actions

Copy link
Copy Markdown

Snapshot Tests
API 23: Report Diff
API 35: Report Diff

@gpeal

gpeal commented Oct 15, 2025

Copy link
Copy Markdown
Collaborator

@hepiao3 Could you add some test animations that validate 3d?

@github-actions

Copy link
Copy Markdown

Snapshot Tests
API 23: Report Diff
API 35: Report Diff

Comment thread snapshot-tests/src/main/assets/Tests/JobCard.json
@github-actions

Copy link
Copy Markdown

Snapshot Tests
API 23: Report Diff
API 35: Report Diff

@gpeal

gpeal commented Oct 22, 2025

Copy link
Copy Markdown
Collaborator

Could you take a look at lottiefiles/animated_laptop_.json? It doesn't look correct in the snapshot tests

@eval3

eval3 commented Oct 23, 2025

Copy link
Copy Markdown
Contributor Author

The files appearing in the report diff all have rx, ry, rz attributes, and now support for these attributes has been added, so there were differences in the comparison during snapshot testing.

@fmdufour

Copy link
Copy Markdown

Plese commit this changes to production. 3d rotation is not working here as well.

@gpeal gpeal left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@hepiao3 Thanks for contributing this!

@gpeal gpeal merged commit 82f1b17 into airbnb:master Oct 23, 2025
7 checks passed
@eval3

eval3 commented Oct 24, 2025

Copy link
Copy Markdown
Contributor Author

@fmdufour Wait a moment, I am currently conducting performance testing.

@msongz

msongz commented Nov 3, 2025

Copy link
Copy Markdown

@hepiao3 hi, any demo could share to see what is supporting?

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

Successfully merging this pull request may close these issues.

4 participants