commit 61e88448d5d32c363cf06a12d3eb81aa763dc4f1
author Gabriel Peal <gpeal@users.noreply.github.com> Sat Apr 23 18:36:09 2022 -0600
committer GitHub <noreply@github.com> Sat Apr 23 17:36:09 2022 -0700
tree 43b906190ad3226b983a5ff93796fb7f4a3b8f20
parent f0ff8a30d265ad883a4cb6a3c77cd0855d5c9cdf

Clear onVisibleAction once the action is taken (#2066)

The root cause of #2053 is that once the fragment was paused and resumed, the onResumeAction was set to RESUME and never cleared. The compass view in the linked project triggered a relayout on every frame which caused the visibility action run over and over again. The animation itself happens to have an empty first frame so it appeared as if the animation disappeared.

Fixes #2053

lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java

diff --git a/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java b/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
index 27d255a..59500a0 100644
--- a/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
+++ b/lottie/src/main/java/com/airbnb/lottie/LottieDrawable.java
@@ -583,6 +583,7 @@
     if (animationsEnabled() || getRepeatCount() == 0) {
       if (isVisible()) {
         animator.playAnimation();
+        onVisibleAction = OnVisibleAction.NONE;
       } else {
         onVisibleAction = OnVisibleAction.PLAY;
       }
@@ -620,6 +621,7 @@
     if (animationsEnabled() || getRepeatCount() == 0) {
       if (isVisible()) {
         animator.resumeAnimation();
+        onVisibleAction = OnVisibleAction.NONE;
       } else {
         onVisibleAction = OnVisibleAction.RESUME;
       }