Prevent touchbar jumps on tap

Author: Nekit678

README.md

@@ -99,8 +99,8 @@ dotnet run --project .\src\Mirabox.Emulator.Panel -c Release
    энкодерам, горизонтальный свайп переключает страницу/сцену. Режим выбирается
    в Stream Dock: у аппаратного вертикального свайпа нет отдельного input-кода,
    которым эмулятор мог бы переключить интерфейс приложения. В Touchbar Mode
-   касание передаётся с момента нажатия: короткий тап запускает элемент, а поток
-   координат горизонтального жеста прокручивает набор.
+   короткий тап передаётся при отпускании и запускает элемент, а после начала
+   горизонтального жеста поток координат прокручивает набор без рывка на клике.
    Панель также распознаёт команду режима и тип загружаемого слоя от Stream Dock.
    Как и на физическом N4 Pro, значки энкодеров в Touchbar Mode скрыты.
 

docs/PROTOCOL.md

@@ -52,11 +52,13 @@ release-пакета физический N4 Pro не формирует. В But
 области отправляют одиночные события `40..43` с `state=00`.
 
 Touch point использует заголовок `ACK 00 ARX`; координаты big-endian находятся
-в байтах 10–13. В Touchbar Mode контроллер отправляет ARX-пакет в начале
-касания и продолжает передавать координаты при движении. Stream Dock распознаёт
-короткий одиночный контакт как нажатие элемента, а последовательность точек —
-как прокрутку. В Button Mode касание обрабатывается как одна из четырёх экранных
-областей, а свайпы влево/вправо имеют hardware code `38`/`39`. У физического N4 Pro
+в байтах 10–13. В Touchbar Mode Stream Dock распознаёт короткий одиночный
+контакт как нажатие элемента, а последовательность точек — как прокрутку.
+Виртуальная панель откладывает одиночную точку до отпускания мыши, а поток
+координат начинает после порога движения, чтобы небольшой дрейф курсора при
+клике не сдвигал содержимое. В Button Mode касание обрабатывается как одна из
+четырёх экранных областей, а свайпы влево/вправо имеют hardware code `38`/`39`.
+У физического N4 Pro
 вертикальный свайп обрабатывается локально прошивкой и отдельного input-кода не
 имеет, поэтому эмулятор не может переключить radio button приложения через HID.
 Режим виртуальной панели выбирается выходной командой `MOD` либо типом слоя,

src/Mirabox.Emulator.Panel/DeviceSurface.cs

@@ -7,6 +7,9 @@ internal sealed class DeviceSurface : Control
 {
     private const int LogicalWidth = 800;
     private const int LogicalHeight = 480;
+    // Ignore ordinary mouse jitter so a click remains a tap instead of
+    // becoming the first step of a scrolling gesture.
+    private const int TouchDragThreshold = 8;
     private const int TouchReportIntervalMs = 16;
     private const float CanvasWidth = 800f;
     private const float CanvasHeight = 420f;
@@ -352,8 +355,6 @@ protected override void OnMouseDown(MouseEventArgs e)
             _touching = true;
             _touchStart = e.Location;
             _activeSecondary = _touchMode == TouchDisplayMode.Button ? SegmentAt(e.Location) : -1;
-            if (_touchMode == TouchDisplayMode.TouchBar)
-                EmitTouch(e.Location, force: true);
             Invalidate();
         }
     }
@@ -368,7 +369,8 @@ protected override void OnMouseMove(MouseEventArgs e)
                 UpdateHover(e.Location);
                 return;
             }
-            if (_touchMode == TouchDisplayMode.TouchBar)
+            if (_touchMode == TouchDisplayMode.TouchBar &&
+                (_lastTouchLocation is not null || StartHorizontalTouchDrag(e.Location)))
                 EmitTouch(e.Location);
         }
         UpdateHover(e.Location);
@@ -399,11 +401,13 @@ protected override void OnMouseUp(MouseEventArgs e)
         {
             if (_touchMode == TouchDisplayMode.TouchBar)
             {
-                // ARX has no separate release flag. The physical controller
-                // reports the contact from its first point through its final
-                // point; Stream Dock derives taps and scrolling from that
-                // sequence and the pause after it.
-                EmitTouch(e.Location, force: true);
+                FlushPendingTouch();
+                if (_lastTouchLocation is null && !HasTouchDragStarted(e.Location))
+                {
+                    // Sending this on MouseDown makes scrollable modules jump.
+                    // Wait until release so a stationary click is one point.
+                    EmitTouch(e.Location, force: true);
+                }
             }
             else
             {
@@ -487,6 +491,26 @@ private int SegmentAt(Point location) =>
     private RectangleF TouchSegment(int index) =>
         new(_touchRect.Left + index * _touchRect.Width / 4, _touchRect.Top, _touchRect.Width / 4, _touchRect.Height);
 
+    private bool HasTouchDragStarted(Point location)
+    {
+        var deltaX = location.X - _touchStart.X;
+        var deltaY = location.Y - _touchStart.Y;
+        return deltaX * deltaX + deltaY * deltaY >= TouchDragThreshold * TouchDragThreshold;
+    }
+
+    private bool StartHorizontalTouchDrag(Point location)
+    {
+        var deltaX = location.X - _touchStart.X;
+        var deltaY = location.Y - _touchStart.Y;
+        if (!HasTouchDragStarted(location) || Math.Abs(deltaX) < Math.Abs(deltaY))
+            return false;
+
+        // Seed the gesture at the real press point only after it has been
+        // classified as a drag, preventing taps from moving the content.
+        EmitTouch(_touchStart, force: true);
+        return true;
+    }
+
     private void EmitTouch(Point location, bool force = false)
     {
         if (_lastTouchLocation == location)