Open alexandrvj opened 5 years ago
Up!
@alexandrvj
issue: I believe the answer to this is very obvious. If you animate a View with built-in classes like ObjectAnimator and the animated View becomes hidden, then the animation will continue regardless. With this library, it is the same.
issue: good finding. I could reproduce the memory leak, a fix will come.
@veghtomi any updates on this fix for the leak, to stop the previous animations ?
There are at least two problem cases with ShimmerLayout animation logic.
<androidx.cardview.widget.CardView android:id="@+id/cardView" android:layout_width="match_parent" android:layout_height="48dp" android:layout_margin="8dp" android:visibility="gone">
</androidx.cardview.widget.CardView>
then our ShimmerLayout will run recursivelly in startShimmerAnimation method, although we have not even shown it to the user.