From e26fe8d017a73e820225b1720179e3d2199602d3 Mon Sep 17 00:00:00 2001
From: dkanada <dkanada@users.noreply.github.com>
Date: Sun, 21 Feb 2021 21:26:06 +0900
Subject: [PATCH] add loading indicator for mini player controls

---
 .../fragments/player/MiniPlayerFragment.java     |  4 ++++
 .../fragment_card_player_playback_controls.xml   | 16 ++++++++--------
 app/src/main/res/layout/fragment_mini_player.xml |  9 +++++----
 3 files changed, 17 insertions(+), 12 deletions(-)

diff --git a/app/src/main/java/com/dkanada/gramophone/fragments/player/MiniPlayerFragment.java b/app/src/main/java/com/dkanada/gramophone/fragments/player/MiniPlayerFragment.java
index ad3dae51..ecb8caf5 100644
--- a/app/src/main/java/com/dkanada/gramophone/fragments/player/MiniPlayerFragment.java
+++ b/app/src/main/java/com/dkanada/gramophone/fragments/player/MiniPlayerFragment.java
@@ -55,11 +55,14 @@ public class MiniPlayerFragment extends AbsMusicServiceFragment implements Music
 
     private void setUpMiniPlayer() {
         setUpPlayPauseButton();
+
         binding.progressBar.setProgressTintList(ColorStateList.valueOf(ThemeStore.accentColor(requireActivity())));
+        binding.progressBar.setIndeterminateTintList(ColorStateList.valueOf(ThemeStore.accentColor(requireActivity())));
     }
 
     private void setUpPlayPauseButton() {
         miniPlayerPlayPauseDrawable = new PlayPauseDrawable(requireActivity());
+
         binding.miniPlayerPlayPauseButton.setImageDrawable(miniPlayerPlayPauseDrawable);
         binding.miniPlayerPlayPauseButton.setColorFilter(ATHUtil.resolveColor(requireActivity(), R.attr.iconColor, ThemeStore.textColorSecondary(requireActivity())), PorterDuff.Mode.SRC_IN);
         binding.miniPlayerPlayPauseButton.setOnClickListener(new PlayPauseButtonOnClickHandler());
@@ -87,6 +90,7 @@ public class MiniPlayerFragment extends AbsMusicServiceFragment implements Music
 
     @Override
     public void onUpdateProgressViews(int progress, int total) {
+        binding.progressBar.setIndeterminate(MusicPlayerRemote.isBuffering());
         binding.progressBar.setMax(total);
         binding.progressBar.setProgress(progress);
     }
diff --git a/app/src/main/res/layout/fragment_card_player_playback_controls.xml b/app/src/main/res/layout/fragment_card_player_playback_controls.xml
index 72574fef..c22099ae 100644
--- a/app/src/main/res/layout/fragment_card_player_playback_controls.xml
+++ b/app/src/main/res/layout/fragment_card_player_playback_controls.xml
@@ -13,14 +13,14 @@
         android:layout_height="@dimen/progress_container_height"
         android:background="@color/twenty_percent_black_overlay">
 
-    <me.zhanghai.android.materialprogressbar.MaterialProgressBar
-        android:id="@+id/player_buffering_indicator"
-        android:layout_width="match_parent"
-        android:layout_height="@dimen/progress_container_height"
-        android:indeterminate="true"
-        app:mpb_progressStyle="horizontal"
-        app:mpb_useIntrinsicPadding="false"
-        style="@style/Widget.MaterialProgressBar.ProgressBar.Horizontal.NoPadding" />
+        <me.zhanghai.android.materialprogressbar.MaterialProgressBar
+            android:id="@+id/player_buffering_indicator"
+            android:layout_width="match_parent"
+            android:layout_height="@dimen/progress_container_height"
+            android:indeterminate="true"
+            app:mpb_progressStyle="horizontal"
+            app:mpb_useIntrinsicPadding="false"
+            style="@style/Widget.MaterialProgressBar.ProgressBar.Horizontal.NoPadding" />
 
     </RelativeLayout>
 
diff --git a/app/src/main/res/layout/fragment_mini_player.xml b/app/src/main/res/layout/fragment_mini_player.xml
index 25d54e80..29063d1e 100644
--- a/app/src/main/res/layout/fragment_mini_player.xml
+++ b/app/src/main/res/layout/fragment_mini_player.xml
@@ -6,7 +6,7 @@
     android:layout_height="@dimen/mini_player_height"
     android:background="?cardBackgroundColor"
     android:clickable="true"
-    android:focusable="false"
+    android:focusable="true"
     android:transitionName="@string/transition_mini_player"
     tools:ignore="UnusedAttribute">
 
@@ -49,11 +49,12 @@
 
     <me.zhanghai.android.materialprogressbar.MaterialProgressBar
         android:id="@+id/progress_bar"
-        style="@style/Widget.MaterialProgressBar.ProgressBar.Horizontal"
         android:layout_width="match_parent"
-        android:layout_height="3dp"
+        android:layout_height="4dp"
         android:layout_gravity="bottom"
         app:mpb_progressStyle="horizontal"
-        app:mpb_useIntrinsicPadding="false" />
+        app:mpb_useIntrinsicPadding="false"
+        app:mpb_setBothDrawables="true"
+        style="@style/Widget.MaterialProgressBar.ProgressBar.Horizontal" />
 
 </FrameLayout>