Simple Menu

Fix select button long press, and add possibility to hold buttons for scrolling

pycardium/modules/py/simple_menu.py

@@ -6,10 +6,22 @@ import time
 import config
 
 TIMEOUT = 0x100
-""":py:func:`~simple_menu.button_events` timeout marker."""
+LONG_PRESS_MS = 1000
+RETRIGGER_MS = 250
+
+try:
+    LONG_PRESS_MS = int(config.get_string("long_press_ms"))
+except OSError:
+    pass
+
+try:
+    RETRIGGER_MS = int(config.get_string("retrigger_ms"))
+except OSError:
+    pass
 
+""":py:func:`~simple_menu.button_events` timeout marker."""
 
-def button_events(timeout=None):
+def button_events(timeout=None, long_press_ms = LONG_PRESS_MS, retrigger_ms = RETRIGGER_MS):
     """
     Iterate over button presses (event-loop).
 
@@ -44,6 +56,8 @@ def button_events(timeout=None):
 
     v = buttons.read(buttons.BOTTOM_LEFT | buttons.BOTTOM_RIGHT | buttons.TOP_RIGHT)
     button_pressed = True if v != 0 else False
+    t0 = time.time_ms()
+    still_pressed = False
 
     if timeout is not None:
         timeout = int(timeout * 1000)
@@ -60,18 +74,28 @@ def button_events(timeout=None):
 
         if v == 0:
             button_pressed = False
-
-        if not button_pressed and v & buttons.BOTTOM_LEFT != 0:
-            button_pressed = True
-            yield buttons.BOTTOM_LEFT
-
-        if not button_pressed and v & buttons.BOTTOM_RIGHT != 0:
-            button_pressed = True
-            yield buttons.BOTTOM_RIGHT
-
-        if not button_pressed and v & buttons.TOP_RIGHT != 0:
-            button_pressed = True
-            yield buttons.TOP_RIGHT
+            still_pressed = False
+        else:
+            if not button_pressed:
+                button_pressed = True
+                t0 = time.time_ms()
+
+                if v & buttons.BOTTOM_LEFT:
+                    yield buttons.BOTTOM_LEFT
+
+                if v & buttons.BOTTOM_RIGHT:
+                    yield buttons.BOTTOM_RIGHT
+
+                if v & buttons.TOP_RIGHT:
+                    yield buttons.TOP_RIGHT
+                
+            if not still_pressed and long_press_ms and time.time_ms() - t0 <= long_press_ms:
+                pass
+            else:
+                if retrigger_ms and time.time_ms() - t0 > retrigger_ms:
+                    button_pressed = False
+                    still_pressed = True
+                    
 
 
 class _ExitMenuException(Exception):
@@ -305,14 +329,14 @@ class Menu:
 
         self.disp.update()
 
-    def run(self):
+    def run(self, long_press_ms = LONG_PRESS_MS, retrigger_ms = RETRIGGER_MS):
         """Start the event-loop."""
         try:
             timeout = self.scroll_speed
             if self.timeout is not None and self.timeout < self.scroll_speed:
                 timeout = self.timeout
 
-            for ev in button_events(timeout):
+            for ev in button_events(timeout, long_press_ms, retrigger_ms):
                 if ev == buttons.BOTTOM_RIGHT:
                     self.select_time = time.time_ms()
                     self.draw_menu(-8)
@@ -336,10 +360,10 @@ class Menu:
                         print("Exception during menu.on_scroll():")
                         sys.print_exception(e)
                 elif ev == self.button_select:
-                    t0 = time.time()
+                    t0 = time.time_ms()
                     long_press = False
-                    while buttons.read(buttons.TOP_RIGHT) > 0:
-                        if time.time() - t0 > 1:
+                    while buttons.read(buttons.button_select) > 0:
+                        if time.time_ms() - t0 > long_press_ms:
                             long_press = True
                             break