maobo/cup_edit
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

move pick to pointerdown for better latency

Browse Source
This commit is contained in:
Nick Fisher
2024-03-04 15:41:35 +08:00
parent 403ea40d09
commit f80d92bf5b
1 changed files with 11 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
lib/widgets/filament_gesture_detector_desktop.dart
View File

@@ -95,8 +95,10 @@ class _FilamentGestureDetectorDesktopState
Widget build(BuildContext context) { Widget build(BuildContext context) {
return Listener( return Listener(
onPointerSignal: (PointerSignalEvent pointerSignal) async { onPointerSignal: (PointerSignalEvent pointerSignal) async {
if (pointerSignal is PointerScrollEvent && widget.enableCamera) { if (pointerSignal is PointerScrollEvent) {
_zoom(pointerSignal); if (widget.enableCamera) {
_zoom(pointerSignal);
}
} else { } else {
throw Exception("TODO"); throw Exception("TODO");
} }
@@ -107,7 +109,13 @@ class _FilamentGestureDetectorDesktopState
// ignore all pointer down events // ignore all pointer down events
// so we can wait to see if the pointer will be held/moved (interpreted as rotate/pan), // so we can wait to see if the pointer will be held/moved (interpreted as rotate/pan),
// or if this is a single mousedown event (interpreted as viewport pick) // or if this is a single mousedown event (interpreted as viewport pick)
onPointerDown: (d) async {}, onPointerDown: (d) async {
if (d.buttons != kTertiaryButton && widget.enablePicking) {
widget.controller
.pick(d.localPosition.dx.toInt(), d.localPosition.dy.toInt());
}
_pointerMoving = false;
},
// holding/moving the left mouse button is interpreted as a pan, middle mouse button as a rotate // holding/moving the left mouse button is interpreted as a pan, middle mouse button as a rotate
onPointerMove: (PointerMoveEvent d) async { onPointerMove: (PointerMoveEvent d) async {
// if this is the first move event, we need to call rotateStart/panStart to set the first coordinates // if this is the first move event, we need to call rotateStart/panStart to set the first coordinates
@@ -139,9 +147,6 @@ class _FilamentGestureDetectorDesktopState
} else { } else {
if (_pointerMoving && widget.enableCamera) { if (_pointerMoving && widget.enableCamera) {
widget.controller.panEnd(); widget.controller.panEnd();
} else if (widget.enablePicking) {
widget.controller
.pick(d.localPosition.dx.toInt(), d.localPosition.dy.toInt());
} }
} }
_pointerMoving = false; _pointerMoving = false;