From 00eefbcf8c9c13b7af5760d8328a309847aec7e7 Mon Sep 17 00:00:00 2001 From: John Crawford Date: Wed, 4 Oct 2023 17:32:00 -0600 Subject: [PATCH 1/5] feat: open battery app when charging --- src/displayapp/DisplayApp.cpp | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp index 3fd34b3a..41a6bf03 100644 --- a/src/displayapp/DisplayApp.cpp +++ b/src/displayapp/DisplayApp.cpp @@ -379,7 +379,16 @@ void DisplayApp::Refresh() { break; case Messages::OnChargingEvent: RestoreBrightness(); - motorController.RunForDuration(15); + if (batteryController.IsCharging() && currentApp == Apps::Clock) { + // Open the battery app if on the clock screen + LoadNewScreen(Apps::BatteryInfo, DisplayApp::FullRefreshDirections::None); + } else if (!batteryController.IsCharging() && currentApp == Apps::BatteryInfo) { + // Close the battery app after being unplugged + LoadNewScreen(Apps::Clock, DisplayApp::FullRefreshDirections::None); + } else { + // Vibrate normally otherwise as to not close any open app + motorController.RunForDuration(15); + } break; } } From 32ca2d35d4a1fd2d93d7d65cc383c73eaf75cf22 Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Mon, 12 Feb 2024 14:08:19 -0500 Subject: [PATCH 2/5] DisplayApp: Add function to determine if app is available --- src/displayapp/DisplayApp.cpp | 7 +++++++ src/displayapp/DisplayApp.h | 1 + 2 files changed, 8 insertions(+) diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp index 41a6bf03..5c42c416 100644 --- a/src/displayapp/DisplayApp.cpp +++ b/src/displayapp/DisplayApp.cpp @@ -564,6 +564,13 @@ void DisplayApp::LoadScreen(Apps app, DisplayApp::FullRefreshDirections directio currentApp = app; } +bool DisplayApp::AppAvailable(Apps app) { + const auto* d = std::find_if(userApps.begin(), userApps.end(), [app](const AppDescription& appDescription) { + return appDescription.app == app; + }); + return d != userApps.end(); +} + void DisplayApp::PushMessage(Messages msg) { if (in_isr()) { BaseType_t xHigherPriorityTaskWoken = pdFALSE; diff --git a/src/displayapp/DisplayApp.h b/src/displayapp/DisplayApp.h index 96bce4dd..55dd94bb 100644 --- a/src/displayapp/DisplayApp.h +++ b/src/displayapp/DisplayApp.h @@ -129,6 +129,7 @@ namespace Pinetime { DisplayApp::FullRefreshDirections nextDirection; System::BootErrors bootError; void ApplyBrightness(); + bool AppAvailable(Apps app); static constexpr size_t returnAppStackSize = 10; Utility::StaticStack returnAppStack; From a403dd343d9eecbb3d2a92c2c5249ad84d037134 Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Tue, 6 Feb 2024 15:41:59 -0500 Subject: [PATCH 3/5] MusicService: Autostart music app on music start When starting music on a paired device, the music service will automatically trigger the music app to open if there is no app already loaded. --- src/components/ble/MusicService.cpp | 5 ++++- src/components/ble/MusicService.h | 8 +++++++- src/components/ble/NimbleController.cpp | 2 +- src/displayapp/DisplayApp.cpp | 8 ++++++++ src/displayapp/Messages.h | 1 + src/systemtask/Messages.h | 1 + src/systemtask/SystemTask.cpp | 3 +++ 7 files changed, 25 insertions(+), 3 deletions(-) diff --git a/src/components/ble/MusicService.cpp b/src/components/ble/MusicService.cpp index 94d97f46..7c04fd9a 100644 --- a/src/components/ble/MusicService.cpp +++ b/src/components/ble/MusicService.cpp @@ -17,6 +17,7 @@ */ #include "components/ble/MusicService.h" #include "components/ble/NimbleController.h" +#include "systemtask/SystemTask.h" #include namespace { @@ -53,7 +54,8 @@ namespace { } } -Pinetime::Controllers::MusicService::MusicService(Pinetime::Controllers::NimbleController& nimble) : nimble(nimble) { +Pinetime::Controllers::MusicService::MusicService(Pinetime::System::SystemTask& systemTask, Pinetime::Controllers::NimbleController& nimble) + : systemTask {systemTask}, nimble(nimble) { characteristicDefinition[0] = {.uuid = &msEventCharUuid.u, .access_cb = MusicCallback, .arg = this, @@ -152,6 +154,7 @@ int Pinetime::Controllers::MusicService::OnCommand(struct ble_gatt_access_ctxt* // These variables need to be updated, because the progress may not be updated immediately, // leading to getProgress() returning an incorrect position. if (playing) { + systemTask.PushMessage(Pinetime::System::Messages::OnMusicStarted); trackProgressUpdateTime = xTaskGetTickCount(); } else { trackProgress += diff --git a/src/components/ble/MusicService.h b/src/components/ble/MusicService.h index f4b902fe..7c16ec0e 100644 --- a/src/components/ble/MusicService.h +++ b/src/components/ble/MusicService.h @@ -27,12 +27,17 @@ #undef min namespace Pinetime { + + namespace System { + class SystemTask; + } + namespace Controllers { class NimbleController; class MusicService { public: - explicit MusicService(NimbleController& nimble); + explicit MusicService(Pinetime::System::SystemTask& systemTask, NimbleController& nimble); void Init(); @@ -87,6 +92,7 @@ namespace Pinetime { bool repeat {false}; bool shuffle {false}; + Pinetime::System::SystemTask& systemTask; NimbleController& nimble; }; } diff --git a/src/components/ble/NimbleController.cpp b/src/components/ble/NimbleController.cpp index 2e7f8003..eb3a84c7 100644 --- a/src/components/ble/NimbleController.cpp +++ b/src/components/ble/NimbleController.cpp @@ -42,7 +42,7 @@ NimbleController::NimbleController(Pinetime::System::SystemTask& systemTask, anService {systemTask, notificationManager}, alertNotificationClient {systemTask, notificationManager}, currentTimeService {dateTimeController}, - musicService {*this}, + musicService {systemTask, *this}, weatherService {dateTimeController}, batteryInformationService {batteryController}, immediateAlertService {systemTask, notificationManager}, diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp index 5c42c416..7617986e 100644 --- a/src/displayapp/DisplayApp.cpp +++ b/src/displayapp/DisplayApp.cpp @@ -259,6 +259,14 @@ void DisplayApp::Refresh() { case Messages::NewNotification: LoadNewScreen(Apps::NotificationsPreview, DisplayApp::FullRefreshDirections::Down); break; + case Messages::MusicStarted: + if (currentApp == Apps::Clock && AppAvailable(Apps::Music)) { + if (state != States::Running) { + PushMessageToSystemTask(System::Messages::GoToRunning); + } + LoadNewScreen(Apps::Music, DisplayApp::FullRefreshDirections::Up); + } + break; case Messages::TimerDone: if (state != States::Running) { PushMessageToSystemTask(System::Messages::GoToRunning); diff --git a/src/displayapp/Messages.h b/src/displayapp/Messages.h index dada3088..1328f363 100644 --- a/src/displayapp/Messages.h +++ b/src/displayapp/Messages.h @@ -15,6 +15,7 @@ namespace Pinetime { ButtonLongerPressed, ButtonDoubleClicked, NewNotification, + MusicStarted, TimerDone, BleFirmwareUpdateStarted, DimScreen, diff --git a/src/systemtask/Messages.h b/src/systemtask/Messages.h index 81be9151..2cb63a39 100644 --- a/src/systemtask/Messages.h +++ b/src/systemtask/Messages.h @@ -24,6 +24,7 @@ namespace Pinetime { OnNewHalfHour, OnChargingEvent, OnPairing, + OnMusicStarted, SetOffAlarm, MeasureBatteryTimerExpired, BatteryPercentageUpdated, diff --git a/src/systemtask/SystemTask.cpp b/src/systemtask/SystemTask.cpp index a56c2591..99986e78 100644 --- a/src/systemtask/SystemTask.cpp +++ b/src/systemtask/SystemTask.cpp @@ -393,6 +393,9 @@ void SystemTask::Work() { nimbleController.DisableRadio(); } break; + case Messages::OnMusicStarted: + displayApp.PushMessage(Pinetime::Applications::Display::Messages::MusicStarted); + break; default: break; } From cb45b9cb37ff18f3f5a0613049155e6314950a1d Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Fri, 9 Feb 2024 13:35:28 -0500 Subject: [PATCH 4/5] NavigationService: Autostart navigation app on nav command When a paired device sends a navigation message, the navigation service will automatically trigger the navigation app to open if there is no app already loaded. --- src/components/ble/NavigationService.cpp | 4 +++- src/components/ble/NavigationService.h | 9 ++++++++- src/components/ble/NimbleController.cpp | 1 + src/displayapp/DisplayApp.cpp | 8 ++++++++ src/displayapp/Messages.h | 1 + src/systemtask/Messages.h | 1 + src/systemtask/SystemTask.cpp | 3 +++ 7 files changed, 25 insertions(+), 2 deletions(-) diff --git a/src/components/ble/NavigationService.cpp b/src/components/ble/NavigationService.cpp index 4922237c..0e6563b2 100644 --- a/src/components/ble/NavigationService.cpp +++ b/src/components/ble/NavigationService.cpp @@ -17,6 +17,7 @@ */ #include "components/ble/NavigationService.h" +#include "systemtask/SystemTask.h" namespace { // 0001yyxx-78fc-48fe-8e23-433b3a1942d0 @@ -43,7 +44,7 @@ namespace { } } // namespace -Pinetime::Controllers::NavigationService::NavigationService() { +Pinetime::Controllers::NavigationService::NavigationService(Pinetime::System::SystemTask& systemTask) : systemTask {systemTask} { characteristicDefinition[0] = {.uuid = &navFlagCharUuid.u, .access_cb = NAVCallback, .arg = this, @@ -96,6 +97,7 @@ int Pinetime::Controllers::NavigationService::OnCommand(struct ble_gatt_access_c } else if (ble_uuid_cmp(ctxt->chr->uuid, &navProgressCharUuid.u) == 0) { m_progress = data[0]; } + systemTask.PushMessage(Pinetime::System::Messages::OnNavChange); } return 0; } diff --git a/src/components/ble/NavigationService.h b/src/components/ble/NavigationService.h index 03e79ac5..bab2cb77 100644 --- a/src/components/ble/NavigationService.h +++ b/src/components/ble/NavigationService.h @@ -27,11 +27,16 @@ #undef min namespace Pinetime { + + namespace System { + class SystemTask; + } + namespace Controllers { class NavigationService { public: - NavigationService(); + explicit NavigationService(Pinetime::System::SystemTask& systemTask); void Init(); @@ -53,6 +58,8 @@ namespace Pinetime { std::string m_narrative; std::string m_manDist; int m_progress; + + Pinetime::System::SystemTask& systemTask; }; } } diff --git a/src/components/ble/NimbleController.cpp b/src/components/ble/NimbleController.cpp index eb3a84c7..bfea93aa 100644 --- a/src/components/ble/NimbleController.cpp +++ b/src/components/ble/NimbleController.cpp @@ -44,6 +44,7 @@ NimbleController::NimbleController(Pinetime::System::SystemTask& systemTask, currentTimeService {dateTimeController}, musicService {systemTask, *this}, weatherService {dateTimeController}, + navService {systemTask}, batteryInformationService {batteryController}, immediateAlertService {systemTask, notificationManager}, heartRateService {*this, heartRateController}, diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp index 7617986e..3e6c0e8f 100644 --- a/src/displayapp/DisplayApp.cpp +++ b/src/displayapp/DisplayApp.cpp @@ -267,6 +267,14 @@ void DisplayApp::Refresh() { LoadNewScreen(Apps::Music, DisplayApp::FullRefreshDirections::Up); } break; + case Messages::NavStarted: + if (currentApp == Apps::Clock && AppAvailable(Apps::Navigation)) { + if (state != States::Running) { + PushMessageToSystemTask(System::Messages::GoToRunning); + } + LoadNewScreen(Apps::Navigation, DisplayApp::FullRefreshDirections::Up); + } + break; case Messages::TimerDone: if (state != States::Running) { PushMessageToSystemTask(System::Messages::GoToRunning); diff --git a/src/displayapp/Messages.h b/src/displayapp/Messages.h index 1328f363..0e21f36d 100644 --- a/src/displayapp/Messages.h +++ b/src/displayapp/Messages.h @@ -16,6 +16,7 @@ namespace Pinetime { ButtonDoubleClicked, NewNotification, MusicStarted, + NavStarted, TimerDone, BleFirmwareUpdateStarted, DimScreen, diff --git a/src/systemtask/Messages.h b/src/systemtask/Messages.h index 2cb63a39..c53d6c58 100644 --- a/src/systemtask/Messages.h +++ b/src/systemtask/Messages.h @@ -25,6 +25,7 @@ namespace Pinetime { OnChargingEvent, OnPairing, OnMusicStarted, + OnNavChange, SetOffAlarm, MeasureBatteryTimerExpired, BatteryPercentageUpdated, diff --git a/src/systemtask/SystemTask.cpp b/src/systemtask/SystemTask.cpp index 99986e78..d814b034 100644 --- a/src/systemtask/SystemTask.cpp +++ b/src/systemtask/SystemTask.cpp @@ -396,6 +396,9 @@ void SystemTask::Work() { case Messages::OnMusicStarted: displayApp.PushMessage(Pinetime::Applications::Display::Messages::MusicStarted); break; + case Messages::OnNavChange: + displayApp.PushMessage(Pinetime::Applications::Display::Messages::NavStarted); + break; default: break; } From 777badf62ced12d63bd08d833583112ff6f8de07 Mon Sep 17 00:00:00 2001 From: Victor Kareh Date: Fri, 9 Feb 2024 13:19:44 -0500 Subject: [PATCH 5/5] settings: Add setting for auto-opening apps Certain apps can be configured to auto-open based on component events: - battery: when charging the watch - music: when playing music from the paired device - navigation: when opening a maps app on the paired device --- src/CMakeLists.txt | 1 + src/components/settings/Settings.h | 18 +++++ src/displayapp/DisplayApp.cpp | 30 +++++--- src/displayapp/apps/Apps.h.in | 1 + src/displayapp/fonts/fonts.json | 2 +- src/displayapp/screens/Symbols.h | 1 + .../screens/settings/SettingAutoOpen.cpp | 75 +++++++++++++++++++ .../screens/settings/SettingAutoOpen.h | 39 ++++++++++ src/displayapp/screens/settings/Settings.h | 1 + src/systemtask/SystemTask.cpp | 8 +- 10 files changed, 164 insertions(+), 12 deletions(-) create mode 100644 src/displayapp/screens/settings/SettingAutoOpen.cpp create mode 100644 src/displayapp/screens/settings/SettingAutoOpen.h diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index fd8ece62..5a3ee306 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -416,6 +416,7 @@ list(APPEND SOURCE_FILES displayapp/screens/settings/SettingSetDate.cpp displayapp/screens/settings/SettingSetTime.cpp displayapp/screens/settings/SettingChimes.cpp + displayapp/screens/settings/SettingAutoOpen.cpp displayapp/screens/settings/SettingShakeThreshold.cpp displayapp/screens/settings/SettingBluetooth.cpp diff --git a/src/components/settings/Settings.h b/src/components/settings/Settings.h index 06312077..e3be110a 100644 --- a/src/components/settings/Settings.h +++ b/src/components/settings/Settings.h @@ -14,6 +14,7 @@ namespace Pinetime { enum class Notification : uint8_t { On, Off, Sleep }; enum class ChimesOption : uint8_t { None, Hours, HalfHours }; enum class WakeUpMode : uint8_t { SingleTap = 0, DoubleTap = 1, RaiseWrist = 2, Shake = 3, LowerWrist = 4 }; + enum class AutoOpen : uint8_t { Battery = 0, Music = 1, Navigation = 2 }; enum class Colors : uint8_t { White, Silver, @@ -253,6 +254,21 @@ namespace Pinetime { return getWakeUpModes()[static_cast(mode)]; } + void SetAutoOpen(AutoOpen app, bool enabled) { + if (enabled != IsAutoOpenOn(app)) { + settingsChanged = true; + } + settings.autoOpen.set(static_cast(app), enabled); + } + + bool IsAutoOpenOn(const AutoOpen app) const { + return GetAutoOpen()[static_cast(app)]; + } + + std::bitset<3> GetAutoOpen() const { + return settings.autoOpen; + } + void SetBrightness(Controllers::BrightnessController::Levels level) { if (level != settings.brightLevel) { settingsChanged = true; @@ -308,6 +324,8 @@ namespace Pinetime { uint16_t shakeWakeThreshold = 150; Controllers::BrightnessController::Levels brightLevel = Controllers::BrightnessController::Levels::Medium; + + std::bitset<3> autoOpen {0b000}; }; SettingsData settings; diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp index 3e6c0e8f..b039c8a7 100644 --- a/src/displayapp/DisplayApp.cpp +++ b/src/displayapp/DisplayApp.cpp @@ -47,6 +47,7 @@ #include "displayapp/screens/settings/SettingSteps.h" #include "displayapp/screens/settings/SettingSetDateTime.h" #include "displayapp/screens/settings/SettingChimes.h" +#include "displayapp/screens/settings/SettingAutoOpen.h" #include "displayapp/screens/settings/SettingShakeThreshold.h" #include "displayapp/screens/settings/SettingBluetooth.h" @@ -395,15 +396,23 @@ void DisplayApp::Refresh() { break; case Messages::OnChargingEvent: RestoreBrightness(); - if (batteryController.IsCharging() && currentApp == Apps::Clock) { - // Open the battery app if on the clock screen - LoadNewScreen(Apps::BatteryInfo, DisplayApp::FullRefreshDirections::None); - } else if (!batteryController.IsCharging() && currentApp == Apps::BatteryInfo) { - // Close the battery app after being unplugged - LoadNewScreen(Apps::Clock, DisplayApp::FullRefreshDirections::None); - } else { - // Vibrate normally otherwise as to not close any open app - motorController.RunForDuration(15); + switch (currentApp) { + case Apps::Clock: + if (batteryController.IsCharging() && settingsController.IsAutoOpenOn(Controllers::Settings::AutoOpen::Battery)) { + // Open the battery app if on the clock screen + LoadNewScreen(Apps::BatteryInfo, DisplayApp::FullRefreshDirections::None); + } + break; + case Apps::BatteryInfo: + if (!batteryController.IsCharging()) { + // Close the battery app after being unplugged + LoadNewScreen(Apps::Clock, DisplayApp::FullRefreshDirections::None); + } + break; + default: + // Vibrate normally otherwise as to not close any open app + motorController.RunForDuration(15); + break; } break; } @@ -543,6 +552,9 @@ void DisplayApp::LoadScreen(Apps app, DisplayApp::FullRefreshDirections directio case Apps::SettingChimes: currentScreen = std::make_unique(settingsController); break; + case Apps::SettingAutoOpen: + currentScreen = std::make_unique(settingsController); + break; case Apps::SettingShakeThreshold: currentScreen = std::make_unique(settingsController, motionController, *systemTask); break; diff --git a/src/displayapp/apps/Apps.h.in b/src/displayapp/apps/Apps.h.in index 2104a267..fe9bff16 100644 --- a/src/displayapp/apps/Apps.h.in +++ b/src/displayapp/apps/Apps.h.in @@ -40,6 +40,7 @@ namespace Pinetime { SettingSteps, SettingSetDateTime, SettingChimes, + SettingAutoOpen, SettingShakeThreshold, SettingBluetooth, Error diff --git a/src/displayapp/fonts/fonts.json b/src/displayapp/fonts/fonts.json index 41c383c0..3dfe79d6 100644 --- a/src/displayapp/fonts/fonts.json +++ b/src/displayapp/fonts/fonts.json @@ -7,7 +7,7 @@ }, { "file": "FontAwesome5-Solid+Brands+Regular.woff", - "range": "0xf294, 0xf242, 0xf54b, 0xf21e, 0xf1e6, 0xf017, 0xf129, 0xf03a, 0xf185, 0xf560, 0xf001, 0xf3fd, 0xf1fc, 0xf45d, 0xf59f, 0xf5a0, 0xf027, 0xf028, 0xf6a9, 0xf04b, 0xf04c, 0xf048, 0xf051, 0xf095, 0xf3dd, 0xf04d, 0xf2f2, 0xf024, 0xf252, 0xf569, 0xf06e, 0xf015, 0xf00c, 0xf0f3, 0xf522, 0xf743" + "range": "0xf294, 0xf242, 0xf54b, 0xf21e, 0xf1e6, 0xf017, 0xf129, 0xf03a, 0xf185, 0xf560, 0xf001, 0xf3fd, 0xf1fc, 0xf45d, 0xf59f, 0xf5a0, 0xf027, 0xf028, 0xf6a9, 0xf04b, 0xf04c, 0xf048, 0xf051, 0xf095, 0xf3dd, 0xf04d, 0xf2f2, 0xf024, 0xf252, 0xf569, 0xf06e, 0xf015, 0xf00c, 0xf0f3, 0xf522, 0xf743, 0xf0ad" } ], "bpp": 1, diff --git a/src/displayapp/screens/Symbols.h b/src/displayapp/screens/Symbols.h index bd958b28..abcc00f4 100644 --- a/src/displayapp/screens/Symbols.h +++ b/src/displayapp/screens/Symbols.h @@ -39,6 +39,7 @@ namespace Pinetime { static constexpr const char* eye = "\xEF\x81\xAE"; static constexpr const char* home = "\xEF\x80\x95"; static constexpr const char* sleep = "\xEE\xBD\x84"; + static constexpr const char* wrench = "\xEF\x82\xAD"; // fontawesome_weathericons.c // static constexpr const char* sun = "\xEF\x86\x85"; diff --git a/src/displayapp/screens/settings/SettingAutoOpen.cpp b/src/displayapp/screens/settings/SettingAutoOpen.cpp new file mode 100644 index 00000000..da9e0c4d --- /dev/null +++ b/src/displayapp/screens/settings/SettingAutoOpen.cpp @@ -0,0 +1,75 @@ +#include "displayapp/screens/settings/SettingAutoOpen.h" +#include +#include "displayapp/DisplayApp.h" +#include "displayapp/screens/Screen.h" +#include "displayapp/screens/Symbols.h" + +using namespace Pinetime::Applications::Screens; + +constexpr std::array SettingAutoOpen::options; + +namespace { + void event_handler(lv_obj_t* obj, lv_event_t event) { + auto* screen = static_cast(obj->user_data); + if (event == LV_EVENT_VALUE_CHANGED) { + screen->UpdateSelected(obj); + } + } +} + +SettingAutoOpen::SettingAutoOpen(Pinetime::Controllers::Settings& settingsController) : settingsController {settingsController} { + lv_obj_t* container1 = lv_cont_create(lv_scr_act(), nullptr); + + lv_obj_set_style_local_bg_opa(container1, LV_CONT_PART_MAIN, LV_STATE_DEFAULT, LV_OPA_TRANSP); + lv_obj_set_style_local_pad_all(container1, LV_CONT_PART_MAIN, LV_STATE_DEFAULT, 10); + lv_obj_set_style_local_pad_inner(container1, LV_CONT_PART_MAIN, LV_STATE_DEFAULT, 5); + lv_obj_set_style_local_border_width(container1, LV_CONT_PART_MAIN, LV_STATE_DEFAULT, 0); + + lv_obj_set_pos(container1, 10, 35); + lv_obj_set_width(container1, LV_HOR_RES - 20); + lv_obj_set_height(container1, LV_VER_RES - 20); + lv_cont_set_layout(container1, LV_LAYOUT_COLUMN_LEFT); + + lv_obj_t* title = lv_label_create(lv_scr_act(), nullptr); + lv_label_set_text_static(title, "Auto Open"); + lv_label_set_align(title, LV_LABEL_ALIGN_CENTER); + lv_obj_align(title, lv_scr_act(), LV_ALIGN_IN_TOP_MID, 15, 15); + + lv_obj_t* icon = lv_label_create(lv_scr_act(), nullptr); + lv_obj_set_style_local_text_color(icon, LV_LABEL_PART_MAIN, LV_STATE_DEFAULT, LV_COLOR_ORANGE); + lv_label_set_text_static(icon, Symbols::wrench); + lv_label_set_align(icon, LV_LABEL_ALIGN_CENTER); + lv_obj_align(icon, title, LV_ALIGN_OUT_LEFT_MID, -10, 0); + + for (unsigned int i = 0; i < options.size(); i++) { + cbOption[i] = lv_checkbox_create(container1, nullptr); + lv_checkbox_set_text(cbOption[i], options[i].name); + if (settingsController.IsAutoOpenOn(static_cast(i))) { + lv_checkbox_set_checked(cbOption[i], true); + } + cbOption[i]->user_data = this; + lv_obj_set_event_cb(cbOption[i], event_handler); + } +} + +SettingAutoOpen::~SettingAutoOpen() { + lv_obj_clean(lv_scr_act()); + settingsController.SaveSettings(); +} + +void SettingAutoOpen::UpdateSelected(lv_obj_t* object) { + // Find the index of the checkbox that triggered the event + for (size_t i = 0; i < options.size(); i++) { + if (cbOption[i] == object) { + bool currentState = settingsController.IsAutoOpenOn(options[i].app); + settingsController.SetAutoOpen(options[i].app, !currentState); + break; + } + } + + // Update checkbox according to current apps. + auto apps = settingsController.GetAutoOpen(); + for (size_t i = 0; i < options.size(); ++i) { + lv_checkbox_set_checked(cbOption[i], apps[i]); + } +} diff --git a/src/displayapp/screens/settings/SettingAutoOpen.h b/src/displayapp/screens/settings/SettingAutoOpen.h new file mode 100644 index 00000000..6d9987de --- /dev/null +++ b/src/displayapp/screens/settings/SettingAutoOpen.h @@ -0,0 +1,39 @@ +#pragma once + +#include +#include +#include + +#include "components/settings/Settings.h" +#include "displayapp/screens/Screen.h" + +namespace Pinetime { + + namespace Applications { + namespace Screens { + + class SettingAutoOpen : public Screen { + public: + SettingAutoOpen(Pinetime::Controllers::Settings& settingsController); + ~SettingAutoOpen() override; + + void UpdateSelected(lv_obj_t* object); + + private: + struct Option { + Controllers::Settings::AutoOpen app; + const char* name; + }; + + Controllers::Settings& settingsController; + static constexpr std::array options = {{ + {Controllers::Settings::AutoOpen::Battery, "Battery"}, + {Controllers::Settings::AutoOpen::Music, "Music"}, + {Controllers::Settings::AutoOpen::Navigation, "Navigation"}, + }}; + + lv_obj_t* cbOption[options.size()]; + }; + } + } +} diff --git a/src/displayapp/screens/settings/Settings.h b/src/displayapp/screens/settings/Settings.h index a21b4ccd..115efecc 100644 --- a/src/displayapp/screens/settings/Settings.h +++ b/src/displayapp/screens/settings/Settings.h @@ -47,6 +47,7 @@ namespace Pinetime { {Symbols::check, "Firmware", Apps::FirmwareValidation}, {Symbols::bluetooth, "Bluetooth", Apps::SettingBluetooth}, + {Symbols::wrench, "Auto Open", Apps::SettingAutoOpen}, {Symbols::list, "About", Apps::SysInfo}, // {Symbols::none, "None", Apps::None}, diff --git a/src/systemtask/SystemTask.cpp b/src/systemtask/SystemTask.cpp index d814b034..02fa9f37 100644 --- a/src/systemtask/SystemTask.cpp +++ b/src/systemtask/SystemTask.cpp @@ -394,10 +394,14 @@ void SystemTask::Work() { } break; case Messages::OnMusicStarted: - displayApp.PushMessage(Pinetime::Applications::Display::Messages::MusicStarted); + if (settingsController.IsAutoOpenOn(Controllers::Settings::AutoOpen::Music)) { + displayApp.PushMessage(Pinetime::Applications::Display::Messages::MusicStarted); + } break; case Messages::OnNavChange: - displayApp.PushMessage(Pinetime::Applications::Display::Messages::NavStarted); + if (settingsController.IsAutoOpenOn(Controllers::Settings::AutoOpen::Navigation)) { + displayApp.PushMessage(Pinetime::Applications::Display::Messages::NavStarted); + } break; default: break;