Mutate the arrow drawable to avoid inconsistent shared states

As the arrow drawable is usually mutated by the animator, it's better to create a copy of its state to avoid animation gliches and arrows in inconsistent states.
arcadefire · May 24, 2019 · 9e18b1f · 9e18b1f
1 parent f62f2fd
commit 9e18b1f
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/library/src/main/java/org/angmarch/views/NiceSpinner.java b/library/src/main/java/org/angmarch/views/NiceSpinner.java
@@ -273,7 +273,8 @@ private Drawable initArrowDrawable(int drawableTint) {
         if (arrowDrawableResId == 0) return null;
         Drawable drawable = ContextCompat.getDrawable(getContext(), arrowDrawableResId);
         if (drawable != null) {
-            drawable = DrawableCompat.wrap(drawable);
+            // Gets a copy of this drawable as this is going to be mutated by the animator
+            drawable = DrawableCompat.wrap(drawable).mutate();
             if (drawableTint != Integer.MAX_VALUE && drawableTint != 0) {
                 DrawableCompat.setTint(drawable, drawableTint);
             }