Make GestureView respond to double touch gesture (jellyfin#1260)

chickdan · Oct 8, 2024 · 071a1f9 · 071a1f9
1 parent c5d6539
commit 071a1f9
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 5 deletions.
diff --git a/Swiftfin/Components/GestureView.swift b/Swiftfin/Components/GestureView.swift
@@ -174,7 +174,7 @@ class UIGestureView: UIView {
 
         let pinchGesture = UIPinchGestureRecognizer(target: self, action: #selector(didPerformPinch))
         let tapGesture = UITapGestureRecognizer(target: self, action: #selector(didPerformTap))
-        let doubleTouchGesture = UITapGestureRecognizer(target: self, action: #selector(didPerformTap))
+        let doubleTouchGesture = UITapGestureRecognizer(target: self, action: #selector(didPerformDoubleTouch))
         doubleTouchGesture.numberOfTouchesRequired = 2
         let longPressGesture = UILongPressGestureRecognizer(target: self, action: #selector(didPerformLongPress))
         longPressGesture.minimumPressDuration = longPressMinimumDuration

diff --git a/Swiftfin/Views/VideoPlayer/VideoPlayer.swift b/Swiftfin/Views/VideoPlayer/VideoPlayer.swift
@@ -376,6 +376,11 @@ extension VideoPlayer {
     }
 
     private func handleDoubleTouchGesture(unitPoint: UnitPoint, taps: Int) {
+        if doubleTouchGesture == .gestureLock {
+            guard !isPresentingOverlay else { return }
+            isGestureLocked.toggle()
+        }
+
         guard !isGestureLocked else {
             updateViewProxy.present(systemName: "lock.fill", title: "Gestures Locked")
             return
@@ -385,10 +390,15 @@ extension VideoPlayer {
         case .none:
             return
         case .aspectFill: ()
-        case .gestureLock:
-            guard !isPresentingOverlay else { return }
-            isGestureLocked.toggle()
-        case .pausePlay: ()
+        case .pausePlay:
+            switch videoPlayerManager.state {
+            case .playing:
+                videoPlayerManager.proxy.pause()
+            default:
+                videoPlayerManager.proxy.play()
+            }
+        default:
+            break
         }
     }
 }