Allow exclusive panning or zooming when pinching #1905

Activity

1. translibrius

   added

   feature

   on Jun 4, 2024
2. translibrius commented on Dec 5, 2024

   

   translibrius

   on Dec 5, 2024

   Author

   More actions

   Hey @JaffaKetchup, sorry to tag you, it's been quite a while since I posted this. Users basically can't use the pinch zoom feature on laptops with no external mouse, which is really frustrating for 'em.

   Do you plan on fixing this anytime soon or should I dip my head into the source and try to do it myself? I see that you are overhauling the gesture system currently so I'm not sure if trying it myself is the best idea since it will be reworked anyway?

   If I should, could you give any pointers of how to get started with this? Thanks 😃
3. JaffaKetchup commented on Dec 5, 2024

   

   JaffaKetchup

   on Dec 5, 2024

   Last edited by JaffaKetchup

   Member

   More actions

   Hey @translibrius,
   Sorry for not responding sooner, we've been letting the gesture issues stack up.

   I'm not sure whether this is a bug or a design question. When using the scroll wheel, or double tapping, the zoom occurs at the pointer location (cursor or tap location). However, it looks like the trackpad is trying to do this (at the cursor location), but it feels kinda broken (as your video shows). I guess the question is whether the trackpad should zoom to the cursor location, or to the center of the map. I'm not sure how to detect whether there is a cursor or not, but maybe that's possible and useful? I'm not quite clear on which you'd prefer?

   The work on gesture and event overhaul has been dormant for a while now, so feel free to have a dig yourself. Most of the code should be in the MapInteractiveViewer - but be warned, there's some amount of confusing methods going on :D. I'll have a quick look as well.

   The code concerned is probably:
   

   flutter_map/lib/src/gestures/map_interactive_viewer.dart

   Lines 575 to 641 in 4dc3bca

   	void _handleScalePinchZoomAndMove(
   	ScaleUpdateDetails details,
   	bool hasPinchZoom,
   	bool hasPinchMove,
   	) {
   	var newCenter = _camera.center;
   	var newZoom = _camera.zoom;
   	// Handle pinch zoom.
   	if (hasPinchZoom && details.scale > 0.0) {
   	newZoom = _getZoomForScale(
   	_mapZoomStart,
   	details.scale + _scaleCorrector,
   	);
   	// Handle starting of pinch zoom.
   	if (!_pinchZoomStarted && newZoom != _mapZoomStart) {
   	_pinchZoomStarted = true;
   	if (!_pinchMoveStarted) {
   	// We want to call moveStart only once for a movement so don't call
   	// it if a pinch move is already underway.
   	widget.controller.moveStarted(MapEventSource.onMultiFinger);
   	}
   	}
   	}
   	// Handle pinch move.
   	if (hasPinchMove) {
   	newCenter = _calculatePinchZoomAndMove(details, newZoom);
   	if (!_pinchMoveStarted && _lastFocalLocal != details.localFocalPoint) {
   	_pinchMoveStarted = true;
   	if (!_pinchZoomStarted) {
   	// We want to call moveStart only once for a movement so don't call
   	// it if a pinch zoom is already underway.
   	widget.controller.moveStarted(MapEventSource.onMultiFinger);
   	}
   	}
   	}
   	if (_pinchZoomStarted || _pinchMoveStarted) {
   	widget.controller.moveRaw(
   	newCenter,
   	newZoom,
   	hasGesture: true,
   	source: MapEventSource.onMultiFinger,
   	);
   	}
   	}
   	LatLng _calculatePinchZoomAndMove(
   	ScaleUpdateDetails details,
   	double zoomAfterPinchZoom,
   	) {
   	final oldCenterPt = _camera.project(_camera.center, zoomAfterPinchZoom);
   	final newFocalLatLong =
   	_camera.offsetToCrs(_focalStartLocal, zoomAfterPinchZoom);
   	final newFocalPt = _camera.project(newFocalLatLong, zoomAfterPinchZoom);
   	final oldFocalPt = _camera.project(_focalStartLatLng, zoomAfterPinchZoom);
   	final zoomDifference = oldFocalPt - newFocalPt;
   	final moveDifference = _rotateOffset(_focalStartLocal - _lastFocalLocal);
   	final newCenterPt = oldCenterPt + zoomDifference + moveDifference.toPoint();
   	return _camera.unproject(newCenterPt, zoomAfterPinchZoom);
   	}

   But I cannot be sure whether it also is used for touchscreen right now - I'll need to check. Indeed, replacing line 640 to oldCenterPt causes zooming into the map center on the trackpad (and maybe other input forms as well). It looks like the calculation here is not quite right for the trackpad (again, maybe its correct for other input methods that use this block).
4. translibrius commented on Dec 6, 2024

   

   translibrius

   on Dec 6, 2024

   Author

   More actions

   Thank you for the detailed response!

   Indeed, I've set the line 640 to _camera.center, and indeed it seems to solve the issue perfectly for me on the trackpad, now it is working as it should. I could very easily just edit this into a fork or something, checking if the platform is windows, apply this workaround, and all my problems are gone, but I would kind of feel bad since this doesn't really solve it at scale.

   I will look into the source more on the weekends in my spare time to try and better understand how it all works under the hood to hopefully make a solution. Perhaps an easy fix would be some interaction flag like FLAG_TRACKPAD_CENTER or something and just return the center if such a flag is set. I'm not sure if it's possible to detect if user is specifically using touchpad pinch as well. The flag could be a simple solution since probably not many people use this package with a trackpad like my app.

   I will let you know if I find something useful about this :)
5. JaffaKetchup

   added

   P: 2 (soon™?)

   S: core

   on Jan 8, 2025
6. JaffaKetchup commented on Jan 8, 2025

   

   JaffaKetchup

   on Jan 8, 2025

   Member

   More actions

   Merged into #2001.
7. JaffaKetchup

   closed this as not plannedon Jan 8, 2025
8. JaffaKetchup

   reopened this on Sep 11, 2025
9. JaffaKetchup

   changed the title [-][FEATURE] Prevent pinch move while pinch zooming[/-] [+][FEATURE] Allow only panning or zooming when pinching[/+] on Sep 11, 2025
10. JaffaKetchup commented on Sep 11, 2025

    

    JaffaKetchup

    on Sep 11, 2025

    Member

    More actions

    Unmerged: #2001 is not directly relevant to this and...

    See #2152 for the bug report which links flutter/flutter#136029. This is the root cause of the issue which triggered this feature request. This feature request should be considered a workaround for that bug, which must be fixed in the Flutter engine.

    The override-trackpad-pinch branch contains a new InteractionOptions.forceOnlySinglePinchGesture property, resolving this request.

    It can be used with the following code, to workaround #2152/the engine issue. This is designed to wrap around the FlutterMap widget. It could be extended to only apply to Windows devices.

    import 'dart:collection';
    
    import 'package:flutter/gestures.dart';
    
    class TrackpadBugDetector extends StatefulWidget {
      const TrackpadBugDetector({super.key, required this.builder});
    
      final Widget Function(BuildContext context, bool bugDetected) builder;
    
      @override
      State<TrackpadBugDetector> createState() => _TrackpadBugDetectorState();
    }
    
    class _TrackpadBugDetectorState extends State<TrackpadBugDetector> {
      // These can be further tuned if necessary
    
      /// Distance (in events) between two events to compare
      static const _numDiffEvents = 5;
    
      /// Absolute difference between two evts on either x/y axis
      static const _panDiffTrigger = 4;
    
      /// Absolute difference between two evts on the scale dimension
      static const _scaleDiffTrigger = 0.01;
    
      final _events = ListQueue<(Offset, double)>(_numDiffEvents);
      bool _latched = false;
    
      @override
      Widget build(BuildContext context) => Listener(
        onPointerPanZoomUpdate: (evt) {
          if (_latched || evt.kind != PointerDeviceKind.trackpad) return;
    
          _events.add((evt.pan, evt.scale));
          if (_events.length > _numDiffEvents) {
            final cmp = _events.removeFirst();
    
            final panDiff = evt.pan - cmp.$1;
            final scaleDiff = (evt.scale - cmp.$2).abs();
    
            if ((panDiff.dx.abs() > _panDiffTrigger ||
                    panDiff.dy.abs() > _panDiffTrigger) &&
                scaleDiff > _scaleDiffTrigger) {
              setState(() => _latched = true);
            }
          }
        },
        child: widget.builder(context, _latched),
      );
    }
11. JaffaKetchup

    changed the title [-][FEATURE] Allow only panning or zooming when pinching[/-] [+]Allow exclusive panning or zooming when pinching[/+] on Sep 11, 2025
12. JaffaKetchup

    removed

    feature

    on Sep 11, 2025
13. JaffaKetchup

    added the

    Feature

    issue type on Sep 11, 2025