From 63e9a3dcc75ae542c77a183df1f2f176c4df6ab4 Mon Sep 17 00:00:00 2001
From: videopixil <videopixil@gmail.com>
Date: Wed, 24 Jun 2026 14:38:26 -0400
Subject: [PATCH 1/2] Fix for hold-to-save

Fix hold-to-save: time the hold from the debounced level, not edge flags

The Button.pressed/released one-shot edge flags intermittently dropped a
press, so hold-to-save would fall through and register as a short tap
(switching mode instead of saving). poll_button now reads the debounced
level (button.value) and detects the press/release edges itself, timing
the hold with time.monotonic(). This also survives the ~300-500 ms loop
stall from the blocking 1 Hz GPS read, which the library's internal hold
timer could not. The debouncer is still used for noise filtering.
---
 Minecraft_Compass/code.py       |  0
 Minecraft_Compass/mc_compass.py | 40 ++++++++++++++++++++++++++-------
 Minecraft_Compass/mc_config.py  |  0
 3 files changed, 32 insertions(+), 8 deletions(-)
 mode change 100644 => 100755 Minecraft_Compass/code.py
 mode change 100644 => 100755 Minecraft_Compass/mc_compass.py
 mode change 100644 => 100755 Minecraft_Compass/mc_config.py

diff --git a/Minecraft_Compass/code.py b/Minecraft_Compass/code.py
old mode 100644
new mode 100755
diff --git a/Minecraft_Compass/mc_compass.py b/Minecraft_Compass/mc_compass.py
old mode 100644
new mode 100755
index 30cfc3005..28ba6bbb6
--- a/Minecraft_Compass/mc_compass.py
+++ b/Minecraft_Compass/mc_compass.py
@@ -17,11 +17,12 @@
 import json
 import math
 import os
+import time
 import microcontroller
 
 from mc_config import (
     ACCEL_SMOOTHING, CARDINALS, DECLINATION_DEFAULT_INDEX, DECLINATION_OPTIONS,
-    EARTH_RADIUS_M, MAX_WAYPOINTS, MODE_COMPASS,
+    EARTH_RADIUS_M, LONG_PRESS_MS, MAX_WAYPOINTS, MODE_COMPASS,
     NVM_DECLINATION_BYTE, NVM_MARINE_BYTE, NVM_ROTATION_BYTE, NVM_THEME_BYTE,
     NVM_UNITS_BYTE, THEMES, WAYPOINTS_FILE,
 )
@@ -321,20 +322,43 @@ def read_mag_calibrated(cal_data):
     )
 
 
+_was_down = [False]     # was the button down on the previous poll?
+_press_start = [None]   # time.monotonic() when it went down, or None
+
+
 def poll_button():
-    """Classify button activity using adafruit_debouncer.Button.
+    """Classify button activity from the debounced level, not edge flags.
 
     Returns one of:
       "none"  - nothing this poll
       "short" - a quick tap (switch mode, or close an open overlay)
-      "save"  - a long press (save a waypoint)
-    The Button class does the press timing; short_duration_ms and
-    long_duration_ms are set where the button is created in code.py.
+      "save"  - a long press held at least LONG_PRESS_MS (save a waypoint)
+
+    adafruit_debouncer keeps a clean debounced level in button.value, but
+    its one-shot edge flags (pressed/released) can miss a fast tap. So this
+    does its own edge detection on the level and times the hold with
+    time.monotonic(). The button is active-low (Pull.UP), so value is False
+    while held. monotonic timing is also immune to the ~300-500 ms stall the
+    blocking GPS read adds to the loop once a second.
     """
     button = hw["button"]
     button.update()
-    if button.long_press:
-        return "save"
-    if button.short_count > 0:
+    down = not button.value   # active-low: value False == pressed
+
+    if down and not _was_down[0]:
+        # Falling edge: the button just went down.
+        _was_down[0] = True
+        _press_start[0] = time.monotonic()
+        return "none"
+
+    if not down and _was_down[0]:
+        # Rising edge: the button was just released. Classify by how long it
+        # was held.
+        _was_down[0] = False
+        start = _press_start[0]
+        _press_start[0] = None
+        if start is not None and (time.monotonic() - start) >= LONG_PRESS_MS / 1000:
+            return "save"
         return "short"
+
     return "none"
diff --git a/Minecraft_Compass/mc_config.py b/Minecraft_Compass/mc_config.py
old mode 100644
new mode 100755

From 92f02debb2356002135030740219c5bb3a1cb5d1 Mon Sep 17 00:00:00 2001
From: videopixil <videopixil@gmail.com>
Date: Wed, 24 Jun 2026 15:43:13 -0400
Subject: [PATCH 2/2] Set file mode to 100644 (remove executable bit)

---
 Minecraft_Compass/code.py       | 0
 Minecraft_Compass/mc_compass.py | 0
 Minecraft_Compass/mc_config.py  | 0
 3 files changed, 0 insertions(+), 0 deletions(-)
 mode change 100755 => 100644 Minecraft_Compass/code.py
 mode change 100755 => 100644 Minecraft_Compass/mc_compass.py
 mode change 100755 => 100644 Minecraft_Compass/mc_config.py

diff --git a/Minecraft_Compass/code.py b/Minecraft_Compass/code.py
old mode 100755
new mode 100644
diff --git a/Minecraft_Compass/mc_compass.py b/Minecraft_Compass/mc_compass.py
old mode 100755
new mode 100644
diff --git a/Minecraft_Compass/mc_config.py b/Minecraft_Compass/mc_config.py
old mode 100755
new mode 100644