Reset timer by long pressing on the button (#1214)

* Reset timer by long pressing on the button

* Consider press_lost as released

Otherwise the bar would keep increasing if the finger slid off the
button

1 files changed, 80 insertions, 20 deletions

diff --git a/src/displayapp/screens/Timer.cpp b/src/displayapp/screens/Timer.cpp
index a1df662a..a25be1c4 100644
--- a/src/displayapp/screens/Timer.cpp
+++ b/src/displayapp/screens/Timer.cpp
@@ -7,7 +7,13 @@ using namespace Pinetime::Applications::Screens;
 
 static void btnEventHandler(lv_obj_t* obj, lv_event_t event) {
   auto* screen = static_cast<Timer*>(obj->user_data);
-  screen->OnButtonEvent(obj, event);
+  if (event == LV_EVENT_PRESSED) {
+    screen->ButtonPressed();
+  } else if (event == LV_EVENT_RELEASED || event == LV_EVENT_PRESS_LOST) {
+    screen->MaskReset();
+  } else if (event == LV_EVENT_SHORT_CLICKED) {
+    screen->ToggleRunning();
+  }
 }
 
 Timer::Timer(DisplayApp* app, Controllers::TimerController& timerController) : Screen(app), timerController {timerController} {
@@ -23,14 +29,37 @@ Timer::Timer(DisplayApp* app, Controllers::TimerController& timerController) : S
   lv_obj_align(minuteCounter.GetObject(), nullptr, LV_ALIGN_IN_TOP_LEFT, 0, 0);
   lv_obj_align(secondCounter.GetObject(), nullptr, LV_ALIGN_IN_TOP_RIGHT, 0, 0);
 
-  btnPlayPause = lv_btn_create(lv_scr_act(), nullptr);
+  highlightObjectMask = lv_objmask_create(lv_scr_act(), nullptr);
+  lv_obj_set_size(highlightObjectMask, 240, 50);
+  lv_obj_align(highlightObjectMask, lv_scr_act(), LV_ALIGN_IN_BOTTOM_MID, 0, 0);
+
+  lv_draw_mask_line_param_t tmpMaskLine;
+
+  lv_draw_mask_line_points_init(&tmpMaskLine, 0, 0, 0, 240, LV_DRAW_MASK_LINE_SIDE_LEFT);
+  highlightMask = lv_objmask_add_mask(highlightObjectMask, &tmpMaskLine);
+
+  lv_obj_t* btnHighlight = lv_obj_create(highlightObjectMask, nullptr);
+  lv_obj_set_style_local_radius(btnHighlight, LV_BTN_PART_MAIN, LV_STATE_DEFAULT, LV_RADIUS_CIRCLE);
+  lv_obj_set_style_local_bg_color(btnHighlight, LV_BTN_PART_MAIN, LV_STATE_DEFAULT, LV_COLOR_ORANGE);
+  lv_obj_set_size(btnHighlight, LV_HOR_RES, 50);
+  lv_obj_align(btnHighlight, lv_scr_act(), LV_ALIGN_IN_BOTTOM_MID, 0, 0);
+
+  btnObjectMask = lv_objmask_create(lv_scr_act(), nullptr);
+  lv_obj_set_size(btnObjectMask, 240, 50);
+  lv_obj_align(btnObjectMask, lv_scr_act(), LV_ALIGN_IN_BOTTOM_MID, 0, 0);
+
+  lv_draw_mask_line_points_init(&tmpMaskLine, 0, 0, 0, 240, LV_DRAW_MASK_LINE_SIDE_RIGHT);
+  btnMask = lv_objmask_add_mask(btnObjectMask, &tmpMaskLine);
+
+  btnPlayPause = lv_btn_create(btnObjectMask, nullptr);
   btnPlayPause->user_data = this;
   lv_obj_set_style_local_radius(btnPlayPause, LV_BTN_PART_MAIN, LV_STATE_DEFAULT, LV_RADIUS_CIRCLE);
   lv_obj_set_style_local_bg_color(btnPlayPause, LV_BTN_PART_MAIN, LV_STATE_DEFAULT, LV_COLOR_MAKE(0x38, 0x38, 0x38));
   lv_obj_set_event_cb(btnPlayPause, btnEventHandler);
   lv_obj_set_size(btnPlayPause, LV_HOR_RES, 50);
-  lv_obj_align(btnPlayPause, lv_scr_act(), LV_ALIGN_IN_BOTTOM_MID, 0, 0);
-  txtPlayPause = lv_label_create(btnPlayPause, nullptr);
+
+  txtPlayPause = lv_label_create(lv_scr_act(), nullptr);
+  lv_obj_align(txtPlayPause, btnPlayPause, LV_ALIGN_CENTER, 0, 0);
 
   if (timerController.IsRunning()) {
     SetTimerRunning();
@@ -46,11 +75,46 @@ Timer::~Timer() {
   lv_obj_clean(lv_scr_act());
 }
 
+void Timer::ButtonPressed() {
+  pressTime = xTaskGetTickCount();
+  buttonPressing = true;
+}
+
+void Timer::MaskReset() {
+  buttonPressing = false;
+  // A click event is processed before a release event,
+  // so the release event would override the "Pause" text without this check
+  if (!timerController.IsRunning()) {
+    lv_label_set_text_static(txtPlayPause, "Start");
+  }
+  maskPosition = 0;
+  UpdateMask();
+}
+
+void Timer::UpdateMask() {
+  lv_draw_mask_line_param_t maskLine;
+
+  lv_draw_mask_line_points_init(&maskLine, maskPosition, 0, maskPosition, 240, LV_DRAW_MASK_LINE_SIDE_LEFT);
+  lv_objmask_update_mask(highlightObjectMask, highlightMask, &maskLine);
+
+  lv_draw_mask_line_points_init(&maskLine, maskPosition, 0, maskPosition, 240, LV_DRAW_MASK_LINE_SIDE_RIGHT);
+  lv_objmask_update_mask(btnObjectMask, btnMask, &maskLine);
+}
+
 void Timer::Refresh() {
   if (timerController.IsRunning()) {
     uint32_t seconds = timerController.GetTimeRemaining() / 1000;
     minuteCounter.SetValue(seconds / 60);
     secondCounter.SetValue(seconds % 60);
+  } else if (buttonPressing && xTaskGetTickCount() > pressTime + pdMS_TO_TICKS(150)) {
+    lv_label_set_text_static(txtPlayPause, "Reset");
+    maskPosition += 15;
+    if (maskPosition > 240) {
+      MaskReset();
+      Reset();
+    } else {
+      UpdateMask();
+    }
   }
 }
 
@@ -66,25 +130,21 @@ void Timer::SetTimerStopped() {
   lv_label_set_text_static(txtPlayPause, "Start");
 }
 
-void Timer::OnButtonEvent(lv_obj_t* obj, lv_event_t event) {
-  if (event == LV_EVENT_CLICKED) {
-    if (obj == btnPlayPause) {
-      if (timerController.IsRunning()) {
-        uint32_t seconds = timerController.GetTimeRemaining() / 1000;
-        minuteCounter.SetValue(seconds / 60);
-        secondCounter.SetValue(seconds % 60);
-        timerController.StopTimer();
-        SetTimerStopped();
-      } else if (secondCounter.GetValue() + minuteCounter.GetValue() > 0) {
-        timerController.StartTimer((secondCounter.GetValue() + minuteCounter.GetValue() * 60) * 1000);
-        Refresh();
-        SetTimerRunning();
-      }
-    }
+void Timer::ToggleRunning() {
+  if (timerController.IsRunning()) {
+    uint32_t seconds = timerController.GetTimeRemaining() / 1000;
+    minuteCounter.SetValue(seconds / 60);
+    secondCounter.SetValue(seconds % 60);
+    timerController.StopTimer();
+    SetTimerStopped();
+  } else if (secondCounter.GetValue() + minuteCounter.GetValue() > 0) {
+    timerController.StartTimer((secondCounter.GetValue() + minuteCounter.GetValue() * 60) * 1000);
+    Refresh();
+    SetTimerRunning();
   }
 }
 
-void Timer::SetDone() {
+void Timer::Reset() {
   minuteCounter.SetValue(0);
   secondCounter.SetValue(0);
   SetTimerStopped();