Merge PR #433 - fix clock face indicators

I misinterpreted the documentation and swapped the alarm and time signal indicators in the clock face. This patch brings it back in sync with the original watch's behavior as described in the manual. Reported-by: 814d3 (GitHub) Reviewed-by: Matheus Afonso Martins Moreira <[email protected]> GitHub-Pull-Request: #433 References: #380 References: #132
joeycastillo · Aug 30, 2024 · 95ca374 · 95ca374
2 parents 657ff72 + f85a7f2
commit 95ca374
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/movement/watch_faces/clock/clock_face.c b/movement/watch_faces/clock/clock_face.c
@@ -70,11 +70,11 @@ static void clock_indicate(WatchIndicatorSegment indicator, bool on) {
 }
 
 static void clock_indicate_alarm(movement_settings_t *settings) {
-    clock_indicate(WATCH_INDICATOR_BELL, settings->bit.alarm_enabled);
+    clock_indicate(WATCH_INDICATOR_SIGNAL, settings->bit.alarm_enabled);
 }
 
 static void clock_indicate_time_signal(clock_state_t *clock) {
-    clock_indicate(WATCH_INDICATOR_SIGNAL, clock->time_signal_enabled);
+    clock_indicate(WATCH_INDICATOR_BELL, clock->time_signal_enabled);
 }
 
 static void clock_indicate_24h(movement_settings_t *settings) {