Properly update colors on theme change

Author: naveensingh

app/src/main/kotlin/com/simplemobiletools/musicplayer/adapters/BaseMusicAdapter.kt

@@ -10,6 +10,7 @@ import com.bumptech.glide.load.resource.bitmap.RoundedCorners
 import com.bumptech.glide.request.RequestOptions
 import com.simplemobiletools.commons.activities.BaseSimpleActivity
 import com.simplemobiletools.commons.adapters.MyRecyclerViewAdapter
+import com.simplemobiletools.commons.extensions.getProperPrimaryColor
 import com.simplemobiletools.commons.helpers.ensureBackgroundThread
 import com.simplemobiletools.commons.views.MyRecyclerView
 import com.simplemobiletools.musicplayer.activities.SimpleControllerActivity
@@ -159,6 +160,14 @@ abstract class BaseMusicAdapter<Type>(
         }
     }
 
+    fun updateColors(newTextColor: Int) {
+        if (textColor != newTextColor || properPrimaryColor != context.getProperPrimaryColor()) {
+            updateTextColor(newTextColor)
+            updatePrimaryColor()
+            notifyDataChanged()
+        }
+    }
+
     @SuppressLint("NotifyDataSetChanged")
     fun notifyDataChanged() = if (itemCount == 0) {
         notifyDataSetChanged()

app/src/main/kotlin/com/simplemobiletools/musicplayer/extensions/LottieAnimationView.kt

@@ -17,11 +17,11 @@ fun LottieAnimationView.updatePlayPauseIcon(isPlaying: Boolean, color: Int) {
             playAnimation()
         }
 
-        addValueCallback(
-            KeyPath("**"),
-            LottieProperty.COLOR_FILTER,
-            { PorterDuffColorFilter(color, PorterDuff.Mode.SRC_IN) }
-        )
         tag = isPlaying
     }
+
+    addValueCallback(
+        KeyPath("**"),
+        LottieProperty.COLOR_FILTER
+    ) { PorterDuffColorFilter(color, PorterDuff.Mode.SRC_IN) }
 }

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/AlbumsFragment.kt

@@ -101,6 +101,7 @@ class AlbumsFragment(context: Context, attributeSet: AttributeSet) : MyViewPager
     override fun setupColors(textColor: Int, adjustedPrimaryColor: Int) {
         binding.albumsPlaceholder.setTextColor(textColor)
         binding.albumsFastscroller.updateColors(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.albumsList.adapter as? AlbumsAdapter

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/ArtistsFragment.kt

@@ -100,6 +100,7 @@ class ArtistsFragment(context: Context, attributeSet: AttributeSet) : MyViewPage
     override fun setupColors(textColor: Int, adjustedPrimaryColor: Int) {
         binding.artistsPlaceholder.setTextColor(textColor)
         binding.artistsFastscroller.updateColors(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.artistsList.adapter as? ArtistsAdapter

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/FoldersFragment.kt

@@ -97,6 +97,7 @@ class FoldersFragment(context: Context, attributeSet: AttributeSet) : MyViewPage
         binding.foldersPlaceholder.setTextColor(textColor)
         binding.foldersFastscroller.updateColors(adjustedPrimaryColor)
         binding.foldersPlaceholder2.setTextColor(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.foldersList.adapter as? FoldersAdapter

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/GenresFragment.kt

@@ -100,6 +100,7 @@ class GenresFragment(context: Context, attributeSet: AttributeSet) : MyViewPager
     override fun setupColors(textColor: Int, adjustedPrimaryColor: Int) {
         binding.genresPlaceholder.setTextColor(textColor)
         binding.genresFastscroller.updateColors(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.genresList.adapter as? GenresAdapter

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/PlaylistsFragment.kt

@@ -107,6 +107,7 @@ class PlaylistsFragment(context: Context, attributeSet: AttributeSet) : MyViewPa
         binding.playlistsPlaceholder.setTextColor(textColor)
         binding.playlistsPlaceholder2.setTextColor(adjustedPrimaryColor)
         binding.playlistsFastscroller.updateColors(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.playlistsList.adapter as? PlaylistsAdapter

app/src/main/kotlin/com/simplemobiletools/musicplayer/fragments/TracksFragment.kt

@@ -107,6 +107,7 @@ class TracksFragment(context: Context, attributeSet: AttributeSet) : MyViewPager
     override fun setupColors(textColor: Int, adjustedPrimaryColor: Int) {
         binding.tracksPlaceholder.setTextColor(textColor)
         binding.tracksFastscroller.updateColors(adjustedPrimaryColor)
+        getAdapter()?.updateColors(textColor)
     }
 
     private fun getAdapter() = binding.tracksList.adapter as? TracksAdapter