tweaks

Signed-off-by: dom <githubaccount56556@proton.me>

lib/common/widgets/gesture/horizontal_drag_gesture_recognizer.dart

@@ -10,6 +10,11 @@ class CustomHorizontalDragGestureRecognizer
   });
 
   Offset? _initialPosition;
+  Offset? get initialPosition => _initialPosition;
+
+  @override
+  DeviceGestureSettings get gestureSettings => _gestureSettings;
+  final _gestureSettings = DeviceGestureSettings(touchSlop: touchSlopH);
 
   @override
   void addAllowedPointer(PointerDownEvent event) {

lib/common/widgets/gesture/image_horizontal_drag_gesture_recognizer.dart

@@ -1,18 +1,17 @@
+import 'package:PiliPlus/common/widgets/gesture/horizontal_drag_gesture_recognizer.dart';
 import 'package:flutter/gestures.dart';
 
 typedef IsBoundaryAllowed =
     bool Function(Offset? initialPosition, OffsetPair lastPosition);
 
 class ImageHorizontalDragGestureRecognizer
-    extends HorizontalDragGestureRecognizer {
+    extends CustomHorizontalDragGestureRecognizer {
   ImageHorizontalDragGestureRecognizer({
     super.debugOwner,
     super.supportedDevices,
     super.allowedButtonsFilter,
   });
 
-  Offset? _initialPosition;
-
   IsBoundaryAllowed? isBoundaryAllowed;
 
   int? _pointer;
@@ -26,20 +25,16 @@ class ImageHorizontalDragGestureRecognizer
     super.addPointer(event);
   }
 
-  @override
-  void addAllowedPointer(PointerDownEvent event) {
-    super.addAllowedPointer(event);
-    _initialPosition = event.position;
-  }
-
   @override
   bool hasSufficientGlobalDistanceToAccept(
     PointerDeviceKind pointerDeviceKind,
     double? deviceTouchSlop,
   ) {
-    return globalDistanceMoved.abs() >
+    return super.hasSufficientGlobalDistanceToAccept(
-            computeHitSlop(pointerDeviceKind, gestureSettings) &&
+          pointerDeviceKind,
-        (isBoundaryAllowed?.call(_initialPosition, lastPosition) ?? true);
+          deviceTouchSlop,
+        ) &&
+        (isBoundaryAllowed?.call(initialPosition, lastPosition) ?? true);
   }
 
   @override

lib/common/widgets/image_viewer/gallery_viewer.dart

@@ -118,8 +118,7 @@ class _GalleryViewerState extends State<GalleryViewer>
     if (PlatformUtils.isDesktop) {
       _tapGestureRecognizer.onSecondaryTapUp = _showDesktopMenu;
     }
-    _horizontalDragGestureRecognizer = ImageHorizontalDragGestureRecognizer()
+    _horizontalDragGestureRecognizer = ImageHorizontalDragGestureRecognizer();
-      ..gestureSettings = gestureSettings;
     _longPressGestureRecognizer = LongPressGestureRecognizer()
       ..onLongPress = _onLongPress
       ..gestureSettings = gestureSettings;

lib/common/widgets/image_viewer/image.dart

@@ -605,8 +605,8 @@ class _ImageState extends State<Image> with WidgetsBindingObserver {
           imgRatio > StyleString.imgMaxRatio &&
           imgHeight > widget.containerSize.height;
       if (isLongPic) {
-        minScale =
+        final compatWidth = math.min(650.0, widget.containerSize.width);
-            widget.containerSize.width / widget.containerSize.height * imgRatio;
+        minScale = compatWidth / widget.containerSize.height * imgRatio;
         maxScale = math.max(widget.maxScale, minScale * 3);
       }
       result = Viewer(

lib/common/widgets/image_viewer/viewer.dart

@@ -17,6 +17,8 @@
 
 import 'dart:math' as math;
 
+import 'package:PiliPlus/common/widgets/gesture/horizontal_drag_gesture_recognizer.dart'
+    show touchSlopH;
 import 'package:PiliPlus/common/widgets/gesture/image_horizontal_drag_gesture_recognizer.dart';
 import 'package:PiliPlus/common/widgets/gesture/image_tap_gesture_recognizer.dart';
 import 'package:PiliPlus/utils/extension/num_ext.dart';
@@ -138,17 +140,16 @@ class _ViewerState extends State<Viewer> with SingleTickerProviderStateMixin {
     _tapGestureRecognizer = widget.tapGestureRecognizer;
     _horizontalDragGestureRecognizer = widget.horizontalDragGestureRecognizer;
 
-    final gestureSettings = MediaQuery.maybeGestureSettingsOf(Get.context!);
     _scaleGestureRecognizer = ScaleGestureRecognizer(debugOwner: this)
       ..dragStartBehavior = .start
       ..onStart = _onScaleStart
       ..onUpdate = _onScaleUpdate
       ..onEnd = _onScaleEnd
-      ..gestureSettings = gestureSettings;
+      ..gestureSettings = DeviceGestureSettings(touchSlop: touchSlopH);
     _doubleTapGestureRecognizer = DoubleTapGestureRecognizer(debugOwner: this)
       ..onDoubleTapDown = _onDoubleTapDown
       ..onDoubleTap = _onDoubleTap
-      ..gestureSettings = gestureSettings;
+      ..gestureSettings = MediaQuery.maybeGestureSettingsOf(Get.context!);
   }
 
   @override
@@ -418,9 +419,9 @@ class _ViewerState extends State<Viewer> with SingleTickerProviderStateMixin {
     final dx = (1 - _scale) * containerWidth / 2;
     final dxOffset = (imageWidth - containerWidth) / 2;
     if (initialPosition.dx < lastPosition.global.dx) {
-      return _position.dx == dx + dxOffset;
+      return _round(_position.dx) == _round(dx + dxOffset);
     } else {
-      return _position.dx == dx - dxOffset;
+      return _round(_position.dx) == _round(dx - dxOffset);
     }
   }
 

lib/pages/main/view.dart

@@ -238,8 +238,6 @@ class _MainAppState extends PopScopeState<MainApp>
   static void _onBack() {
     if (Platform.isAndroid) {
       Utils.channel.invokeMethod('back');
-    } else {
-      SystemNavigator.pop();
     }
   }