From 98ac447bdbadc65b3c7cc1ee9bddee327bd41b6c Mon Sep 17 00:00:00 2001
From: Tim Keller <geekboy1011@gmail.com>
Date: Mon, 4 Oct 2021 17:11:27 +0000
Subject: [PATCH] Fixed button color changing

---
 .../screens/settings/SettingShakeThreshold.cpp  | 17 +++++++++--------
 .../screens/settings/SettingShakeThreshold.h    |  1 -
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/src/displayapp/screens/settings/SettingShakeThreshold.cpp b/src/displayapp/screens/settings/SettingShakeThreshold.cpp
index aaabce02..c42bb9ae 100644
--- a/src/displayapp/screens/settings/SettingShakeThreshold.cpp
+++ b/src/displayapp/screens/settings/SettingShakeThreshold.cpp
@@ -7,7 +7,7 @@
 using namespace Pinetime::Applications::Screens;
 
 namespace {
-  static void event_handler(lv_obj_t* obj, lv_event_t event) {
+  void event_handler(lv_obj_t* obj, lv_event_t event) {
     SettingShakeThreshold* screen = static_cast<SettingShakeThreshold*>(obj->user_data);
     screen->UpdateSelected(obj, event);
   }
@@ -41,10 +41,11 @@ SettingShakeThreshold::SettingShakeThreshold(DisplayApp* app,
   lv_obj_set_height(animArc, lv_obj_get_height(positionArc));
   lv_obj_align_mid(animArc, positionArc, LV_ALIGN_CENTER, 0, 0);
   lv_obj_set_style_local_line_opa(animArc, LV_ARC_PART_BG, LV_STATE_DEFAULT, 0);
-  lv_obj_set_style_local_line_opa(animArc, LV_ARC_PART_INDIC, LV_STATE_DEFAULT, LV_OPA_60);
+  lv_obj_set_style_local_line_opa(animArc, LV_ARC_PART_INDIC, LV_STATE_DEFAULT, LV_OPA_70);
   lv_obj_set_style_local_line_opa(animArc, LV_ARC_PART_KNOB, LV_STATE_DEFAULT, LV_OPA_0);
   lv_obj_set_style_local_line_color(animArc, LV_ARC_PART_INDIC, LV_STATE_DEFAULT, LV_COLOR_RED);
   lv_obj_set_style_local_bg_color(animArc, LV_ARC_PART_BG, LV_STATE_CHECKED, LV_COLOR_TRANSP);
+
   animArc->user_data = this;
   lv_obj_set_click(animArc, false);
 
@@ -80,8 +81,8 @@ void SettingShakeThreshold::Refresh() {
     if (xTaskGetTickCount() - vCalTime > pdMS_TO_TICKS(2000)) {
       vCalTime = xTaskGetTickCount();
       calibrating = 2;
-      lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_BTN_STATE_CHECKED_RELEASED, LV_COLOR_RED);
-      lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_BTN_STATE_CHECKED_PRESSED, LV_COLOR_RED);
+      lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_STATE_CHECKED, LV_COLOR_RED);
+      lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_STATE_CHECKED, LV_COLOR_RED);
       lv_label_set_text(calLabel, "Shake!!");
     }
   }
@@ -108,17 +109,17 @@ void SettingShakeThreshold::UpdateSelected(lv_obj_t* object, lv_event_t event) {
   switch (event) {
     case LV_EVENT_VALUE_CHANGED: {
       if (object == calButton) {
-
         if (lv_btn_get_state(calButton) == LV_BTN_STATE_CHECKED_RELEASED && calibrating == 0) {
           lv_arc_set_value(positionArc, 0);
           calibrating = 1;
           vCalTime = xTaskGetTickCount();
           lv_label_set_text(calLabel, "Ready!");
-          lv_obj_set_click(calButton,false);
+          lv_obj_set_click(positionArc, false);
+          lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_STATE_CHECKED, LV_COLOR_GREEN);
+          lv_obj_set_style_local_bg_color(calButton, LV_BTN_PART_MAIN, LV_STATE_CHECKED, LV_COLOR_GREEN);
         } else if (lv_btn_get_state(calButton) == LV_BTN_STATE_RELEASED) {
-
           calibrating = 0;
-          lv_obj_set_click(calButton,true);
+          lv_obj_set_click(positionArc, true);
           lv_label_set_text(calLabel, "Calibrate");
         }
         break;
diff --git a/src/displayapp/screens/settings/SettingShakeThreshold.h b/src/displayapp/screens/settings/SettingShakeThreshold.h
index ed565892..6d4ccf66 100644
--- a/src/displayapp/screens/settings/SettingShakeThreshold.h
+++ b/src/displayapp/screens/settings/SettingShakeThreshold.h
@@ -27,7 +27,6 @@ namespace Pinetime {
         System::SystemTask& systemTask;
         uint8_t calibrating;
         uint32_t vDecay,vCalTime;
-        lv_obj_t* cbOption[2];
         lv_obj_t *positionArc, *animArc,*calButton, *calLabel;
         lv_task_t* refreshTask;
       };