video: fixed gesture handling

lib/widgets/common/magnifier/core/core.dart

@@ -19,6 +19,7 @@ class MagnifierCore extends StatefulWidget {
   final bool applyScale;
   final double panInertia;
   final MagnifierTapCallback? onTap;
+  final MagnifierDoubleTapCallback? onDoubleTap;
   final Widget child;
 
   const MagnifierCore({
@@ -27,7 +28,8 @@ class MagnifierCore extends StatefulWidget {
     required this.scaleStateCycle,
     required this.applyScale,
     this.panInertia = .2,
-    required this.onTap,
+    this.onTap,
+    this.onDoubleTap,
     required this.child,
   }) : super(key: key);
 
@@ -204,6 +206,12 @@ class _MagnifierCoreState extends State<MagnifierCore> with TickerProviderStateM
 
   void onDoubleTap(TapDownDetails details) {
     final viewportTapPosition = details.localPosition;
+    if (widget.onDoubleTap != null) {
+      final viewportSize = scaleBoundaries.viewportSize;
+      final alignment = Alignment(viewportTapPosition.dx / viewportSize.width, viewportTapPosition.dy / viewportSize.height);
+      if (widget.onDoubleTap?.call(alignment) == true) return;
+    }
+
     final childTapPosition = scaleBoundaries.viewportToChildPosition(controller, viewportTapPosition);
     nextScaleState(ChangeSource.gesture, childFocalPoint: childTapPosition);
   }

lib/widgets/common/magnifier/magnifier.dart

@@ -29,6 +29,7 @@ class Magnifier extends StatelessWidget {
     this.scaleStateCycle = defaultScaleStateCycle,
     this.applyScale = true,
     this.onTap,
+    this.onDoubleTap,
     required this.child,
   }) : super(key: key);
 
@@ -49,6 +50,7 @@ class Magnifier extends StatelessWidget {
   final ScaleStateCycle scaleStateCycle;
   final bool applyScale;
   final MagnifierTapCallback? onTap;
+  final MagnifierDoubleTapCallback? onDoubleTap;
   final Widget child;
 
   @override
@@ -68,6 +70,7 @@ class Magnifier extends StatelessWidget {
           scaleStateCycle: scaleStateCycle,
           applyScale: applyScale,
           onTap: onTap,
+          onDoubleTap: onDoubleTap,
           child: child,
         );
       },
@@ -81,3 +84,7 @@ typedef MagnifierTapCallback = Function(
   MagnifierState state,
   Offset childTapPosition,
 );
+
+typedef MagnifierDoubleTapCallback = bool Function(
+  Alignment alignment,
+);

lib/widgets/viewer/video_action_delegate.dart

@@ -57,10 +57,10 @@ class VideoActionDelegate with FeedbackMixin, PermissionAwareMixin, SizeAwareMix
         _togglePlayPause(context, controller);
         break;
       case EntryAction.videoReplay10:
-        if (controller.isReady) controller.seekTo(controller.currentPosition - 10000);
+        controller.seekTo(controller.currentPosition - 10000);
         break;
       case EntryAction.videoSkip10:
-        if (controller.isReady) controller.seekTo(controller.currentPosition + 10000);
+        controller.seekTo(controller.currentPosition + 10000);
         break;
       case EntryAction.open:
         final entry = controller.entry;

lib/widgets/viewer/visual/entry_page_view.dart

@@ -30,6 +30,7 @@ import 'package:collection/collection.dart';
 import 'package:decorated_icon/decorated_icon.dart';
 import 'package:flutter/material.dart';
 import 'package:provider/provider.dart';
+import 'package:tuple/tuple.dart';
 
 class EntryPageView extends StatefulWidget {
   final AvesEntry mainEntry, pageEntry;
@@ -173,7 +174,7 @@ class _EntryPageViewState extends State<EntryPageView> {
   }
 
   Widget _buildSvgView() {
-    var child = _buildMagnifier(
+    return _buildMagnifier(
       maxScale: const ScaleLevel(factor: 25),
       scaleStateCycle: _vectorScaleStateCycle,
       applyScale: false,
@@ -186,25 +187,25 @@ class _EntryPageViewState extends State<EntryPageView> {
         ),
       ),
     );
-    return child;
   }
 
   Widget _buildVideoView() {
     final videoController = context.read<VideoConductor>().getController(entry);
     if (videoController == null) return const SizedBox();
 
-    Positioned _buildDoubleTapDetector(
+    return ValueListenableBuilder<double>(
-      EntryAction action, {
+      valueListenable: videoController.sarNotifier,
-      double widthFactor = 1,
+      builder: (context, sar, child) {
-      AlignmentGeometry alignment = Alignment.center,
+        final videoDisplaySize = entry.videoDisplaySize(sar);
-      IconData? Function()? icon,
+
-    }) {
+        return Selector<Settings, Tuple2<bool, bool>>(
-      return Positioned.fill(
+          selector: (context, s) => Tuple2(s.videoGestureDoubleTapTogglePlay, s.videoGestureSideDoubleTapSeek),
-        child: FractionallySizedBox(
+          builder: (context, s, child) {
-          alignment: alignment,
+            final playGesture = s.item1;
-          widthFactor: widthFactor,
+            final seekGesture = s.item2;
-          child: GestureDetector(
+            final useActionGesture = playGesture || seekGesture;
-            onDoubleTap: () {
+
+            void _applyAction(EntryAction action, {IconData? Function()? icon}) {
               _actionFeedbackChildNotifier.value = DecoratedIcon(
                 icon?.call() ?? action.getIconData(),
                 size: 48,
@@ -219,19 +220,31 @@ class _EntryPageViewState extends State<EntryPageView> {
                 controller: videoController,
                 action: action,
               ).dispatch(context);
-            },
-          ),
-        ),
-      );
             }
 
-    return ValueListenableBuilder<double>(
+            MagnifierDoubleTapCallback? _onDoubleTap = useActionGesture
-      valueListenable: videoController.sarNotifier,
+                ? (alignment) {
-      builder: (context, sar, child) {
+                    final x = alignment.x;
-        final videoDisplaySize = entry.videoDisplaySize(sar);
+                    if (seekGesture) {
-        final playGesture = settings.videoGestureDoubleTapTogglePlay;
+                      if (x < .25) {
-        final seekGesture = settings.videoGestureSideDoubleTapSeek;
+                        _applyAction(EntryAction.videoReplay10);
-        final useActionGesture = playGesture || seekGesture;
+                        return true;
+                      } else if (x > .75) {
+                        _applyAction(EntryAction.videoSkip10);
+                        return true;
+                      }
+                    }
+                    if (playGesture) {
+                      _applyAction(
+                        EntryAction.videoTogglePlay,
+                        icon: () => videoController.isPlaying ? AIcons.pause : AIcons.play,
+                      );
+                      return true;
+                    }
+                    return false;
+                  }
+                : null;
+
             return Stack(
               fit: StackFit.expand,
               children: [
@@ -239,6 +252,7 @@ class _EntryPageViewState extends State<EntryPageView> {
                   children: [
                     _buildMagnifier(
                       displaySize: videoDisplaySize,
+                      onDoubleTap: _onDoubleTap,
                       child: VideoView(
                         entry: entry,
                         controller: videoController,
@@ -254,15 +268,6 @@ class _EntryPageViewState extends State<EntryPageView> {
                         viewStateNotifier: _viewStateNotifier,
                         debugMode: true,
                       ),
-                if (playGesture)
-                  _buildDoubleTapDetector(
-                    EntryAction.videoTogglePlay,
-                    icon: () => videoController.isPlaying ? AIcons.pause : AIcons.play,
-                  ),
-                if (seekGesture) ...[
-                  _buildDoubleTapDetector(EntryAction.videoReplay10, widthFactor: .25, alignment: Alignment.centerLeft),
-                  _buildDoubleTapDetector(EntryAction.videoSkip10, widthFactor: .25, alignment: Alignment.centerRight),
-                ],
                     if (useActionGesture)
                       ValueListenableBuilder<Widget?>(
                         valueListenable: _actionFeedbackChildNotifier,
@@ -272,14 +277,24 @@ class _EntryPageViewState extends State<EntryPageView> {
                       ),
                   ],
                 ),
-            _buildVideoCover(videoController, videoDisplaySize),
+                _buildVideoCover(
+                  videoController: videoController,
+                  videoDisplaySize: videoDisplaySize,
+                  onDoubleTap: _onDoubleTap,
+                ),
               ],
             );
           },
         );
+      },
+    );
   }
 
-  StreamBuilder<VideoStatus> _buildVideoCover(AvesVideoController videoController, Size videoDisplaySize) {
+  StreamBuilder<VideoStatus> _buildVideoCover({
+    required AvesVideoController videoController,
+    required Size videoDisplaySize,
+    required MagnifierDoubleTapCallback? onDoubleTap,
+  }) {
     // fade out image to ease transition with the player
     return StreamBuilder<VideoStatus>(
       stream: videoController.statusStream,
@@ -307,6 +322,7 @@ class _EntryPageViewState extends State<EntryPageView> {
                   return _buildMagnifier(
                     controller: coverController,
                     displaySize: coverSize,
+                    onDoubleTap: onDoubleTap,
                     child: Image(
                       image: videoCoverUriImage,
                     ),
@@ -342,6 +358,7 @@ class _EntryPageViewState extends State<EntryPageView> {
     ScaleLevel maxScale = maxScale,
     ScaleStateCycle scaleStateCycle = defaultScaleStateCycle,
     bool applyScale = true,
+    MagnifierDoubleTapCallback? onDoubleTap,
     required Widget child,
   }) {
     return Magnifier(
@@ -355,6 +372,7 @@ class _EntryPageViewState extends State<EntryPageView> {
       scaleStateCycle: scaleStateCycle,
       applyScale: applyScale,
       onTap: (c, d, s, o) => _onTap(),
+      onDoubleTap: onDoubleTap,
       child: child,
     );
   }