From e8e24574b7231cb3b63e8ef932ed61a2be528f19 Mon Sep 17 00:00:00 2001
From: thyttan <6uuxstm66@mozmail.com⁩>
Date: Mon, 23 Sep 2024 22:23:15 +0200
Subject: [PATCH] edgeclk: refactor step event handling

Only register event handler if we want to actually use the event.
---
 apps/edgeclk/app.js | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/apps/edgeclk/app.js b/apps/edgeclk/app.js
index 085291bf8..79310c3da 100644
--- a/apps/edgeclk/app.js
+++ b/apps/edgeclk/app.js
@@ -281,10 +281,9 @@
   };
 
   const onStep = function () {
-    if (settings.redrawOnStep) drawSteps();
+    drawSteps();
   }
 
-
   /* Lifecycle Functions
   ------------------------------------------------------------------------------*/
 
@@ -305,7 +304,7 @@
     Bangle.on('charging', onCharging);
 
     // Continously update step count when they happen:
-    Bangle.on('step', onStep);
+    if (settings.redrawOnStep) Bangle.on('step', onStep);
   };
 
   const deregisterEvents = function () {
@@ -314,7 +313,7 @@
     Bangle.removeListener('health', onHealth);
     Bangle.removeListener('lock', onLock);
     Bangle.removeListener('charging', onCharging);
-    Bangle.removeListener('step', onStep);
+    if (settings.redrawOnStep) Bangle.removeListener('step', onStep);
   };
 
   const startTimers = function () {