Skip to content

Add 3D rotation support for Lottie animations#2666

Merged
gpeal merged 3 commits intoairbnb:masterfrom
hepiao3:3d-layers
Oct 23, 2025
Merged

Add 3D rotation support for Lottie animations#2666
gpeal merged 3 commits intoairbnb:masterfrom
hepiao3:3d-layers

Conversation

@hepiao3
Copy link
Copy Markdown
Contributor

@hepiao3 hepiao3 commented Oct 9, 2025

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

@hepiao3
Copy link
Copy Markdown
Contributor Author

hepiao3 commented Oct 9, 2025

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
Copy link
Copy Markdown
Collaborator

gpeal commented Oct 15, 2025

@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

@github-actions
Copy link
Copy Markdown

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

@gpeal
Copy link
Copy Markdown
Collaborator

gpeal commented Oct 22, 2025

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

@hepiao3
Copy link
Copy Markdown
Contributor Author

hepiao3 commented Oct 23, 2025

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.

Copy link
Copy Markdown
Collaborator

@gpeal gpeal left a comment

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
@hepiao3
Copy link
Copy Markdown
Contributor Author

hepiao3 commented Oct 24, 2025

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

@msongz
Copy link
Copy Markdown

msongz commented Nov 3, 2025

@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