opt image gesture

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

lib/common/widgets/custom_tooltip.dart

@@ -252,12 +252,7 @@ class _RenderToolTip extends RenderBox
 
   @override
   void paint(PaintingContext context, Offset offset) {
-    RenderBox? child = firstChild;
+    defaultPaint(context, offset);
-    while (child != null) {
-      final childParentData = child.parentData as MultiChildLayoutParentData;
-      context.paintChild(child, childParentData.offset + offset);
-      child = childParentData.nextSibling;
-    }
   }
 }
 

lib/common/widgets/image/custom_grid_view.dart

@@ -30,14 +30,16 @@ import 'package:PiliPlus/utils/image_utils.dart';
 import 'package:PiliPlus/utils/page_utils.dart';
 import 'package:PiliPlus/utils/platform_utils.dart';
 import 'package:PiliPlus/utils/storage_pref.dart';
-import 'package:flutter/material.dart'
+import 'package:flutter/gestures.dart'
-    hide CustomMultiChildLayout, MultiChildLayoutDelegate;
+    show TapGestureRecognizer, LongPressGestureRecognizer;
+import 'package:flutter/material.dart';
 import 'package:flutter/rendering.dart'
     show
         ContainerRenderObjectMixin,
         RenderBoxContainerDefaultsMixin,
         MultiChildLayoutParentData,
-        BoxHitTestResult;
+        BoxHitTestResult,
+        BoxHitTestEntry;
 import 'package:flutter/services.dart' show HapticFeedback;
 import 'package:get/get_core/src/get_main.dart';
 import 'package:get/get_navigation/get_navigation.dart';
@@ -149,8 +151,9 @@ class CustomGridView extends StatelessWidget {
 
   static bool enableImgMenu = Pref.enableImgMenu;
 
-  void _showMenu(BuildContext context, Offset offset, ImageModel item) {
+  void _showMenu(BuildContext context, int index, Offset offset) {
     HapticFeedback.mediumImpact();
+    final item = picArr[index];
     showMenu(
       context: context,
       position: PageUtils.menuPosition(offset),
@@ -254,6 +257,13 @@ class CustomGridView extends StatelessWidget {
           column: column,
           width: imageWidth,
           height: imageHeight,
+          onTap: (index) => onTap(context, index),
+          onSecondaryTapUp: enableImgMenu && PlatformUtils.isDesktop
+              ? (index, offset) => _showMenu(context, index, offset)
+              : null,
+          onLongPressStart: enableImgMenu && PlatformUtils.isMobile
+              ? (index, offset) => _showMenu(context, index, offset)
+              : null,
           children: List.generate(length, (index) {
             final item = picArr[index];
             final borderRadius = _borderRadius(column, length, index);
@@ -293,18 +303,7 @@ class CustomGridView extends StatelessWidget {
             }
             return LayoutId(
               id: index,
-              child: GestureDetector(
-                onTap: () => onTap(context, index),
-                onSecondaryTapUp: enableImgMenu && PlatformUtils.isDesktop
-                    ? (details) =>
-                          _showMenu(context, details.globalPosition, item)
-                    : null,
-                onLongPressStart: enableImgMenu && PlatformUtils.isMobile
-                    ? (details) =>
-                          _showMenu(context, details.globalPosition, item)
-                    : null,
               child: child,
-              ),
             );
           }),
         ),
@@ -321,12 +320,18 @@ class ImageGrid extends MultiChildRenderObjectWidget {
     required this.column,
     required this.width,
     required this.height,
+    required this.onTap,
+    required this.onSecondaryTapUp,
+    required this.onLongPressStart,
   });
 
   final double space;
   final int column;
   final double width;
   final double height;
+  final ValueChanged<int> onTap;
+  final OnShowMenu? onSecondaryTapUp;
+  final OnShowMenu? onLongPressStart;
 
   @override
   RenderObject createRenderObject(BuildContext context) {
@@ -335,6 +340,9 @@ class ImageGrid extends MultiChildRenderObjectWidget {
       column: column,
       width: width,
       height: height,
+      onTap: onTap,
+      onSecondaryTapUp: onSecondaryTapUp,
+      onLongPressStart: onLongPressStart,
     );
   }
 
@@ -344,10 +352,15 @@ class ImageGrid extends MultiChildRenderObjectWidget {
       ..space = space
       ..column = column
       ..width = width
-      ..height = height;
+      ..height = height
+      ..onTap = onTap
+      ..onSecondaryTapUp = onSecondaryTapUp
+      ..onLongPressStart = onLongPressStart;
   }
 }
 
+typedef OnShowMenu = Function(int index, Offset offset);
+
 class RenderImageGrid extends RenderBox
     with
         ContainerRenderObjectMixin<RenderBox, MultiChildLayoutParentData>,
@@ -357,10 +370,54 @@ class RenderImageGrid extends RenderBox
     required int column,
     required double width,
     required double height,
+    required ValueChanged<int> onTap,
+    required OnShowMenu? onSecondaryTapUp,
+    required OnShowMenu? onLongPressStart,
   }) : _space = space,
        _column = column,
        _width = width,
-       _height = height;
+       _height = height,
+       _onTap = onTap,
+       _onSecondaryTapUp = onSecondaryTapUp,
+       _onLongPressStart = onLongPressStart {
+    _tapGestureRecognizer = TapGestureRecognizer()..onTap = _handleOnTap;
+    if (onSecondaryTapUp != null) {
+      _tapGestureRecognizer.onSecondaryTapUp = _handleSecondaryTapUp;
+    }
+    if (onLongPressStart != null) {
+      _longPressGestureRecognizer = LongPressGestureRecognizer()
+        ..onLongPressStart = _handleLongPressStart;
+    }
+  }
+
+  ValueChanged<int> _onTap;
+  set onTap(ValueChanged<int> value) {
+    _onTap = value;
+  }
+
+  OnShowMenu? _onSecondaryTapUp;
+  set onSecondaryTapUp(OnShowMenu? value) {
+    _onSecondaryTapUp = value;
+  }
+
+  OnShowMenu? _onLongPressStart;
+  set onLongPressStart(OnShowMenu? value) {
+    _onLongPressStart = value;
+  }
+
+  int? _index;
+
+  void _handleOnTap() {
+    _onTap(_index!);
+  }
+
+  void _handleSecondaryTapUp(TapUpDetails details) {
+    _onSecondaryTapUp!(_index!, details.globalPosition);
+  }
+
+  void _handleLongPressStart(LongPressStartDetails details) {
+    _onLongPressStart!(_index!, details.globalPosition);
+  }
 
   double _space;
   double get space => _space;
@@ -426,17 +483,60 @@ class RenderImageGrid extends RenderBox
 
   @override
   void paint(PaintingContext context, Offset offset) {
-    RenderBox? child = firstChild;
+    defaultPaint(context, offset);
-    while (child != null) {
-      final childParentData = child.parentData as MultiChildLayoutParentData;
-      context.paintChild(child, childParentData.offset + offset);
-      child = childParentData.nextSibling;
-    }
   }
 
   @override
   bool hitTestChildren(BoxHitTestResult result, {required Offset position}) {
-    return defaultHitTestChildren(result, position: position);
+    RenderBox? child = lastChild;
+    while (child != null) {
+      final childParentData = child.parentData as MultiChildLayoutParentData;
+      final bool isHit = result.addWithPaintOffset(
+        offset: childParentData.offset,
+        position: position,
+        hitTest: (BoxHitTestResult result, Offset transformed) {
+          assert(transformed == position - childParentData.offset);
+          if (child!.size.contains(transformed)) {
+            result.add(BoxHitTestEntry(child, transformed));
+            return true;
+          }
+          return false;
+        },
+      );
+      if (isHit) {
+        _index = childParentData.id as int;
+        return true;
+      }
+      child = childParentData.previousSibling;
+    }
+    _index = null;
+    return false;
+  }
+
+  @override
+  void handleEvent(PointerEvent event, BoxHitTestEntry entry) {
+    if (event is PointerDownEvent) {
+      _tapGestureRecognizer.addPointer(event);
+      _longPressGestureRecognizer?.addPointer(event);
+    }
+  }
+
+  late final TapGestureRecognizer _tapGestureRecognizer;
+  LongPressGestureRecognizer? _longPressGestureRecognizer;
+
+  @override
+  void dispose() {
+    _tapGestureRecognizer
+      ..onTap = null
+      ..onSecondaryTapUp = null
+      ..dispose();
+    _longPressGestureRecognizer
+      ?..onLongPressStart = null
+      ..dispose();
+    _longPressGestureRecognizer = null;
+    _onSecondaryTapUp = null;
+    _onLongPressStart = null;
+    super.dispose();
   }
 
   @override

lib/common/widgets/player_bar.dart

@@ -90,12 +90,7 @@ class RenderBottomBar extends RenderBox
   }
 
   void _paintChild(PaintingContext context, Offset offset) {
-    RenderBox? child = firstChild;
+    defaultPaint(context, offset);
-    while (child != null) {
-      final childParentData = child.parentData as MultiChildLayoutParentData;
-      context.paintChild(child, childParentData.offset + offset);
-      child = childParentData.nextSibling;
-    }
   }
 
   @override