From 45235acd5d91497adf9cdf5b078b85b5b4f16b73 Mon Sep 17 00:00:00 2001
From: vfsfitvnm <vfsfitvnm@protonmail.com>
Date: Sun, 4 Sep 2022 20:45:41 +0200
Subject: [PATCH] Remove "loop none" option (#276)

---
 .../vimusic/service/PlayerService.kt          |  5 ++-
 .../vimusic/ui/views/player/Controls.kt       | 24 ++++-------
 app/src/main/res/drawable/infinite.xml        | 18 ++++++++
 app/src/main/res/drawable/repeat.xml          | 34 ---------------
 app/src/main/res/drawable/repeat_one.xml      | 41 -------------------
 5 files changed, 30 insertions(+), 92 deletions(-)
 create mode 100644 app/src/main/res/drawable/infinite.xml
 delete mode 100644 app/src/main/res/drawable/repeat.xml
 delete mode 100644 app/src/main/res/drawable/repeat_one.xml

diff --git a/app/src/main/kotlin/it/vfsfitvnm/vimusic/service/PlayerService.kt b/app/src/main/kotlin/it/vfsfitvnm/vimusic/service/PlayerService.kt
index 76d81f1..6be4b1b 100644
--- a/app/src/main/kotlin/it/vfsfitvnm/vimusic/service/PlayerService.kt
+++ b/app/src/main/kotlin/it/vfsfitvnm/vimusic/service/PlayerService.kt
@@ -194,7 +194,10 @@ class PlayerService : InvincibleService(), Player.Listener, PlaybackStatsListene
             .setUsePlatformDiagnostics(false)
             .build()
 
-        player.repeatMode = preferences.getInt(repeatModeKey, Player.REPEAT_MODE_OFF)
+        player.repeatMode = when (preferences.getInt(repeatModeKey, Player.REPEAT_MODE_ALL)) {
+            Player.REPEAT_MODE_ONE -> Player.REPEAT_MODE_ONE
+            else -> Player.REPEAT_MODE_ALL
+        }
         player.skipSilenceEnabled = preferences.getBoolean(skipSilenceKey, false)
         player.addListener(this)
         player.addAnalyticsListener(PlaybackStatsListener(false, this))
diff --git a/app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/views/player/Controls.kt b/app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/views/player/Controls.kt
index 3d76b21..361ed45 100644
--- a/app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/views/player/Controls.kt
+++ b/app/src/main/kotlin/it/vfsfitvnm/vimusic/ui/views/player/Controls.kt
@@ -249,29 +249,21 @@ fun Controls(
             )
 
             Image(
-                painter = painterResource(
-                    if (repeatMode == Player.REPEAT_MODE_ONE) {
-                        R.drawable.repeat_one
-                    } else {
-                        R.drawable.repeat
-                    }
-                ),
+                painter = painterResource(R.drawable.infinite),
                 contentDescription = null,
                 colorFilter = ColorFilter.tint(
-                    if (repeatMode == Player.REPEAT_MODE_OFF) {
-                        colorPalette.textDisabled
-                    } else {
+                    if (repeatMode == Player.REPEAT_MODE_ONE) {
                         colorPalette.text
+                    } else {
+                        colorPalette.textDisabled
                     }
                 ),
                 modifier = Modifier
                     .clickable {
-                        binder.player.repeatMode
-                            .plus(2)
-                            .mod(3)
-                            .let {
-                                binder.player.repeatMode = it
-                            }
+                        binder.player.repeatMode = when (binder.player.repeatMode) {
+                            Player.REPEAT_MODE_ONE -> Player.REPEAT_MODE_ALL
+                            else -> Player.REPEAT_MODE_ONE
+                        }
                     }
                     .weight(1f)
                     .size(24.dp)
diff --git a/app/src/main/res/drawable/infinite.xml b/app/src/main/res/drawable/infinite.xml
new file mode 100644
index 0000000..b8444f3
--- /dev/null
+++ b/app/src/main/res/drawable/infinite.xml
@@ -0,0 +1,18 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="24dp"
+    android:height="24dp"
+    android:viewportWidth="512"
+    android:viewportHeight="512">
+  <path
+      android:pathData="M256,256s-48,-96 -126,-96c-54.12,0 -98,43 -98,96s43.88,96 98,96c30,0 56.45,-13.18 78,-32"
+      android:strokeWidth="36"
+      android:fillColor="#00000000"
+      android:strokeColor="#000"
+      android:strokeLineCap="round"/>
+  <path
+      android:pathData="M256,256s48,96 126,96c54.12,0 98,-43 98,-96s-43.88,-96 -98,-96c-29.37,0 -56.66,13.75 -78,32"
+      android:strokeWidth="36"
+      android:fillColor="#00000000"
+      android:strokeColor="#000"
+      android:strokeLineCap="round"/>
+</vector>
diff --git a/app/src/main/res/drawable/repeat.xml b/app/src/main/res/drawable/repeat.xml
deleted file mode 100644
index 65e522f..0000000
--- a/app/src/main/res/drawable/repeat.xml
+++ /dev/null
@@ -1,34 +0,0 @@
-<vector xmlns:android="http://schemas.android.com/apk/res/android"
-    android:width="24dp"
-    android:height="24dp"
-    android:viewportWidth="512"
-    android:viewportHeight="512">
-  <path
-      android:pathData="M320,120l48,48l-48,48"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M352,168H144a80.24,80.24 0,0 0,-80 80v16"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M192,392l-48,-48l48,-48"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M160,344H368a80.24,80.24 0,0 0,80 -80V248"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-</vector>
diff --git a/app/src/main/res/drawable/repeat_one.xml b/app/src/main/res/drawable/repeat_one.xml
deleted file mode 100644
index 26013d7..0000000
--- a/app/src/main/res/drawable/repeat_one.xml
+++ /dev/null
@@ -1,41 +0,0 @@
-<vector xmlns:android="http://schemas.android.com/apk/res/android"
-    android:width="24dp"
-    android:height="24dp"
-    android:viewportWidth="512"
-    android:viewportHeight="512">
-  <path
-      android:pathData="M320,120l48,48l-48,48"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M352,168H144a80.24,80.24 0,0 0,-80 80v16"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M192,392l-48,-48l48,-48"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="M160,344H353.113"
-      android:strokeLineJoin="round"
-      android:strokeWidth="32"
-      android:fillColor="#00000000"
-      android:strokeColor="#000000"
-      android:strokeLineCap="round"/>
-  <path
-      android:pathData="m446.293,243.326 l-48.027,30.375v0l44.094,-21.604v95.885H448V248v0,-2.322 -2.352z"
-      android:strokeLineJoin="round"
-      android:strokeWidth="28.021"
-      android:fillColor="#ff0000"
-      android:strokeColor="#000000"
-      android:strokeLineCap="round"/>
-</vector>