From 0ecf6d4c0eade032644f0c41882a740fe437b584 Mon Sep 17 00:00:00 2001
From: peerless2012 <peerless2012@users.noreply.github.com>
Date: Mon, 22 Jan 2024 22:31:50 +0800
Subject: [PATCH] feat: add long press for 2x speed (#596)

* Add long press for 2x speed.

* Fix resource hard code warn.

* Fix lint warn.

* refactor: use `player.setPlaybackSpeed`

* refactor: make playback speed increase easily adjustable later

---------

Co-authored-by: jarnedemeulemeester <jarnedemeulemeester@gmail.com>
---
 .../jellyfin/utils/PlayerGestureHelper.kt     | 20 ++++++++++++
 .../src/main/res/layout/activity_player.xml   | 31 +++++++++++++++++++
 .../main/res/drawable/ic_speed_forward.xml    | 27 ++++++++++++++++
 3 files changed, 78 insertions(+)
 create mode 100644 core/src/main/res/drawable/ic_speed_forward.xml

diff --git a/app/phone/src/main/java/dev/jdtech/jellyfin/utils/PlayerGestureHelper.kt b/app/phone/src/main/java/dev/jdtech/jellyfin/utils/PlayerGestureHelper.kt
index 8c11ba3a..9591abef 100644
--- a/app/phone/src/main/java/dev/jdtech/jellyfin/utils/PlayerGestureHelper.kt
+++ b/app/phone/src/main/java/dev/jdtech/jellyfin/utils/PlayerGestureHelper.kt
@@ -55,6 +55,9 @@ class PlayerGestureHelper(
 
     private var lastScaleEvent: Long = 0
 
+    private var playbackSpeedIncrease: Float = 2f
+    private var lastPlaybackSpeed: Float = 0f
+
     private val screenWidth = Resources.getSystem().displayMetrics.widthPixels
     private val screenHeight = Resources.getSystem().displayMetrics.heightPixels
 
@@ -69,6 +72,18 @@ class PlayerGestureHelper(
                 return true
             }
 
+            @SuppressLint("SetTextI18n")
+            override fun onLongPress(e: MotionEvent) {
+                playerView.player?.let {
+                    if (it.isPlaying) {
+                        lastPlaybackSpeed = it.playbackParameters.speed
+                        it.setPlaybackSpeed(playbackSpeedIncrease)
+                        activity.binding.gestureSpeedText.text = playbackSpeedIncrease.toString() + "x"
+                        activity.binding.gestureSpeedLayout.visibility = View.VISIBLE
+                    }
+                }
+            }
+
             override fun onDoubleTap(e: MotionEvent): Boolean {
                 // Disables double tap gestures if view is locked
                 if (isControlsLocked) return false
@@ -362,6 +377,11 @@ class PlayerGestureHelper(
                 }
             }
         }
+        if (lastPlaybackSpeed > 0 && (event.action == MotionEvent.ACTION_UP || event.action == MotionEvent.ACTION_CANCEL)) {
+            playerView.player?.setPlaybackSpeed(lastPlaybackSpeed)
+            lastPlaybackSpeed = 0f
+            activity.binding.gestureSpeedLayout.visibility = View.GONE
+        }
     }
 
     private fun longToTimestamp(duration: Long, noSign: Boolean = false): String {
diff --git a/app/phone/src/main/res/layout/activity_player.xml b/app/phone/src/main/res/layout/activity_player.xml
index ee2e8977..c2c1f5b7 100644
--- a/app/phone/src/main/res/layout/activity_player.xml
+++ b/app/phone/src/main/res/layout/activity_player.xml
@@ -113,6 +113,37 @@
             tools:ignore="ContentDescription" />
     </LinearLayout>
 
+    <LinearLayout
+        android:id="@+id/gesture_speed_layout"
+        android:layout_width="wrap_content"
+        android:layout_height="64dp"
+        android:layout_gravity="center_horizontal|top"
+        android:layout_margin="16dp"
+        android:background="@drawable/overlay_background"
+        android:clickable="false"
+        android:gravity="center_vertical"
+        android:orientation="horizontal"
+        android:visibility="gone"
+        tools:visibility="visible">
+
+        <TextView
+            android:id="@+id/gesture_speed_text"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginStart="16dp"
+            android:gravity="center"
+            android:textSize="20sp"
+            android:textColor="@android:color/white" />
+
+        <ImageView
+            android:id="@+id/gesture_speed_image"
+            android:layout_width="36dp"
+            android:layout_height="24dp"
+            android:layout_marginHorizontal="16dp"
+            android:src="@drawable/ic_speed_forward"
+            tools:ignore="ContentDescription" />
+    </LinearLayout>
+
     <ImageView
         android:id="@+id/image_ffwd_animation_ripple"
         android:layout_width="50dp"
diff --git a/core/src/main/res/drawable/ic_speed_forward.xml b/core/src/main/res/drawable/ic_speed_forward.xml
new file mode 100644
index 00000000..58a7a17d
--- /dev/null
+++ b/core/src/main/res/drawable/ic_speed_forward.xml
@@ -0,0 +1,27 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"
+    android:width="36dp"
+    android:height="24dp"
+    android:viewportWidth="36"
+    android:viewportHeight="24">
+  <path
+      android:pathData="M14,19l9,-7l-9,-7l0,14z"
+      android:strokeLineJoin="round"
+      android:strokeWidth="2"
+      android:fillColor="#00000000"
+      android:strokeColor="@android:color/white"
+      android:strokeLineCap="round"/>
+  <path
+      android:pathData="M2,19l9,-7l-9,-7l0,14z"
+      android:strokeLineJoin="round"
+      android:strokeWidth="2"
+      android:fillColor="#00000000"
+      android:strokeColor="@android:color/white"
+      android:strokeLineCap="round"/>
+  <path
+      android:pathData="M26,19l9,-7l-9,-7l0,14z"
+      android:strokeLineJoin="round"
+      android:strokeWidth="2"
+      android:fillColor="#00000000"
+      android:strokeColor="@android:color/white"
+      android:strokeLineCap="round"/>
+</vector>