M menubar/main.swift → menubar/main.swift

@@ -408,6 +408,8 @@ let importanceExpected = ["blocked low", "done new high", "done old", "done older low", "working high", "working new"]
     if importanceListed != importanceExpected {
         failures.append("order: status first, then importance, then time: \(importanceListed)")
     }
+    if isDrag(from: .zero, to: NSPoint(x: 2, y: 3)) { failures.append("click: a few points of jitter is a click") }
+    if !isDrag(from: .zero, to: NSPoint(x: 0, y: 5)) { failures.append("click: moving 5 points is a drag") }
     let high = Row(paneId: "%1", agent: "pi", name: "Fix it", status: .done, session: "s", window: "w", since: 1, importance: .high)
     if rowTitle(high).string != "✓  ! Fix it" || rowTitlePrefix(high) != "✓  ! " {
         failures.append("title: a high agent is marked: \(rowTitle(high).string)")

M menubar/panel.swift → menubar/panel.swift

@@ -194,10 +194,17 @@ ])
     return effect
 }
 
+/// Whether the pointer moved far enough from where the button went down to
+/// be a drag: a click on a trackpad or mouse moves it a point or two.
+func isDrag(from down: NSPoint, to now: NSPoint) -> Bool {
+    hypot(now.x - down.x, now.y - down.y) > 4
+}
+
 /// A click target that does not swallow drags: a drag moves the panel.
 class ClickView: NSView {
     var onClick: () -> Void = {}
     private var dragged = false
+    private var downAt = NSPoint.zero
     var hovered = false { didSet { needsDisplay = true; hoverChanged() } }
 
     func hoverChanged() {}
@@ -222,10 +229,13 @@ }
 
     override func mouseEntered(with event: NSEvent) { hovered = true }
     override func mouseExited(with event: NSEvent) { hovered = false }
-    override func mouseDown(with event: NSEvent) { dragged = false }
+    override func mouseDown(with event: NSEvent) {
+        dragged = false
+        downAt = NSEvent.mouseLocation
+    }
 
     override func mouseDragged(with event: NSEvent) {
-        if !dragged {
+        if !dragged, isDrag(from: downAt, to: NSEvent.mouseLocation) {
             dragged = true
             window?.performDrag(with: event)
         }