diff options
| author | Kevin O'Connor <kevin@koconnor.net> | 2020-03-15 12:15:15 -0400 |
|---|---|---|
| committer | Kevin O'Connor <kevin@koconnor.net> | 2020-06-15 17:23:34 -0400 |
| commit | 8d9566b945bb6f58b10d915a32ce6a00de674953 (patch) | |
| tree | ceeaa7480e1be9c102cb849e617f98802666f5d6 /klippy/extras/display | |
| parent | 7d8ade74e885c6cd141046dd55cf82d1d5b4fdf5 (diff) | |
| download | kutter-8d9566b945bb6f58b10d915a32ce6a00de674953.tar.gz kutter-8d9566b945bb6f58b10d915a32ce6a00de674953.tar.xz kutter-8d9566b945bb6f58b10d915a32ce6a00de674953.zip | |
menu_keys: Reschedule long-click timer on each click
Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
Diffstat (limited to 'klippy/extras/display')
| -rw-r--r-- | klippy/extras/display/menu_keys.py | 35 |
1 files changed, 15 insertions, 20 deletions
diff --git a/klippy/extras/display/menu_keys.py b/klippy/extras/display/menu_keys.py index b99c1076..73fcfb35 100644 --- a/klippy/extras/display/menu_keys.py +++ b/klippy/extras/display/menu_keys.py @@ -11,6 +11,7 @@ TIMER_DELAY = .200 class MenuKeys: def __init__(self, config, callback): self.printer = config.get_printer() + self.reactor = self.printer.get_reactor() self.callback = callback # buttons self.encoder_pins = config.get('encoder_pins', None) @@ -30,7 +31,6 @@ class MenuKeys: 'analog_range_down_pin', None) self.analog_range_kill_pin = config.get( 'analog_range_kill_pin', None) - self._last_click_press = 0 self.analog_pullup = config.getfloat( 'analog_pullup_resistor', 4700., above=0.) self._encoder_fast_rate = config.getfloat( @@ -120,18 +120,14 @@ class MenuKeys: else: self.buttons.register_button_push( self.kill_pin, self.kill_callback) - # start timer - reactor = self.printer.get_reactor() - reactor.register_timer(self.timer_event, reactor.NOW) + # long click timer + self.is_short_click = False + self.click_timer = self.reactor.register_timer(self.long_click_event) - def timer_event(self, eventtime): - # check long press - if (self._last_click_press > 0 and ( - eventtime - self._last_click_press) >= LONG_PRESS_DURATION): - # long click - self._last_click_press = 0 - self.callback('long_click', eventtime) - return eventtime + TIMER_DELAY + def long_click_event(self, eventtime): + self.is_short_click = False + self.callback('long_click', eventtime) + return self.reactor.NEVER # buttons & encoder callbacks def encoder_cw_callback(self, eventtime): @@ -153,14 +149,13 @@ class MenuKeys: self.callback('down', eventtime) def click_callback(self, eventtime, state): - if self.click_pin: - if state: - self._last_click_press = eventtime - elif self._last_click_press > 0: - if (eventtime - self._last_click_press) < LONG_PRESS_DURATION: - # short click - self._last_click_press = 0 - self.callback('click', eventtime) + if state: + self.is_short_click = True + self.reactor.update_timer(self.click_timer, + eventtime + LONG_PRESS_DURATION) + elif self.is_short_click: + self.reactor.update_timer(self.click_timer, self.reactor.NEVER) + self.callback('click', eventtime) def back_callback(self, eventtime): if self.back_pin: |