From 197247feddc9d775373f65d24f00577e069ea8e4 Mon Sep 17 00:00:00 2001
From: Bnyro <bnyro@tutanota.com>
Date: Mon, 30 Oct 2023 12:15:10 +0100
Subject: [PATCH] refactor: move reverse queue button to sort dialog

---
 .../github/libretube/ui/sheets/PlayingQueueSheet.kt    |  9 +++------
 app/src/main/res/drawable/ic_reverse.xml               | 10 ----------
 app/src/main/res/layout/queue_bottom_sheet.xml         |  6 ------
 3 files changed, 3 insertions(+), 22 deletions(-)
 delete mode 100644 app/src/main/res/drawable/ic_reverse.xml

diff --git a/app/src/main/java/com/github/libretube/ui/sheets/PlayingQueueSheet.kt b/app/src/main/java/com/github/libretube/ui/sheets/PlayingQueueSheet.kt
index eb9ba9de5..ec0e2551a 100644
--- a/app/src/main/java/com/github/libretube/ui/sheets/PlayingQueueSheet.kt
+++ b/app/src/main/java/com/github/libretube/ui/sheets/PlayingQueueSheet.kt
@@ -54,11 +54,6 @@ class PlayingQueueSheet : ExpandedBottomSheet() {
             AddToPlaylistDialog().show(childFragmentManager, null)
         }
 
-        binding.reverse.setOnClickListener {
-            PlayingQueue.setStreams(PlayingQueue.getStreams().reversed())
-            adapter.notifyDataSetChanged()
-        }
-
         binding.repeat.setOnClickListener {
             // select the next available repeat mode
             PlayingQueue.repeatMode = when (PlayingQueue.repeatMode) {
@@ -135,7 +130,8 @@ class PlayingQueueSheet : ExpandedBottomSheet() {
             R.string.creation_date,
             R.string.most_views,
             R.string.uploader_name,
-            R.string.shuffle
+            R.string.shuffle,
+            R.string.tooltip_reverse
         )
             .map { requireContext().getString(it) }
             .toTypedArray()
@@ -160,6 +156,7 @@ class PlayingQueueSheet : ExpandedBottomSheet() {
                             .filter { it !in toShuffle }
                             .plus(toShuffle.shuffled())
                     }
+                    4 -> PlayingQueue.getStreams().reversed()
                     else -> throw IllegalArgumentException()
                 }
                 PlayingQueue.setStreams(newQueue)
diff --git a/app/src/main/res/drawable/ic_reverse.xml b/app/src/main/res/drawable/ic_reverse.xml
deleted file mode 100644
index 1daf24465..000000000
--- a/app/src/main/res/drawable/ic_reverse.xml
+++ /dev/null
@@ -1,10 +0,0 @@
-<vector xmlns:android="http://schemas.android.com/apk/res/android"
-    android:width="24dp"
-    android:height="24dp"
-    android:tint="?attr/colorControlNormal"
-    android:viewportWidth="48"
-    android:viewportHeight="48">
-    <path
-        android:fillColor="#FF000000"
-        android:pathData="m16.25,41.4 l-2.1,-2.1 3.2,-3.3q-5.5,0 -9.425,-3.75Q4,28.5 4,23q0,-5.35 3.725,-9.175Q11.45,10 16.8,10h7.6v3h-7.6q-4.1,0 -6.95,2.925Q7,18.85 7,23q0,4.25 3.125,7.125T17.6,33l-3.3,-3.3 2.1,-2.1 6.8,6.85ZM28.6,36v-3L44,33v3ZM28.6,24.5v-3L44,21.5v3ZM27.4,13v-3L44,10v3Z" />
-</vector>
diff --git a/app/src/main/res/layout/queue_bottom_sheet.xml b/app/src/main/res/layout/queue_bottom_sheet.xml
index 47e697431..e84d6d62c 100644
--- a/app/src/main/res/layout/queue_bottom_sheet.xml
+++ b/app/src/main/res/layout/queue_bottom_sheet.xml
@@ -43,12 +43,6 @@
                     style="@style/QueueSheetOption"
                     android:src="@drawable/ic_repeat" />
 
-                <ImageView
-                    android:id="@+id/reverse"
-                    android:tooltipText="@string/tooltip_reverse"
-                    style="@style/QueueSheetOption"
-                    android:src="@drawable/ic_reverse" />
-
                 <ImageView
                     android:id="@+id/add_to_playlist"
                     android:tooltipText="@string/addToPlaylist"