From 842357c0b0c42c2c1310c64a7baec41bd2cff58d Mon Sep 17 00:00:00 2001 From: Riku Isokoski Date: Fri, 20 May 2022 11:55:40 +0300 Subject: Automatically create enough screens for all apps and settings diff --git a/src/displayapp/screens/ApplicationList.cpp b/src/displayapp/screens/ApplicationList.cpp index 385b7ec..f04b9ef 100644 --- a/src/displayapp/screens/ApplicationList.cpp +++ b/src/displayapp/screens/ApplicationList.cpp @@ -1,13 +1,23 @@ #include "displayapp/screens/ApplicationList.h" #include #include -#include "displayapp/screens/Symbols.h" -#include "displayapp/screens/Tile.h" #include "displayapp/Apps.h" #include "displayapp/DisplayApp.h" using namespace Pinetime::Applications::Screens; +constexpr std::array ApplicationList::applications; + +auto ApplicationList::CreateScreenList() const { + std::array()>, nScreens> screens; + for (size_t i = 0; i < screens.size(); i++) { + screens[i] = [this, i]() -> std::unique_ptr { + return CreateScreen(i); + }; + } + return screens; +} + ApplicationList::ApplicationList(Pinetime::Applications::DisplayApp* app, Pinetime::Controllers::Settings& settingsController, Pinetime::Controllers::Battery& batteryController, @@ -16,14 +26,7 @@ ApplicationList::ApplicationList(Pinetime::Applications::DisplayApp* app, settingsController {settingsController}, batteryController {batteryController}, dateTimeController {dateTimeController}, - screens {app, - settingsController.GetAppMenu(), - { - [this]() -> std::unique_ptr { return CreateScreen1(); }, - [this]() -> std::unique_ptr { return CreateScreen2(); }, - [this]() -> std::unique_ptr { return CreateScreen3(); } - }, - Screens::ScreenListModes::UpDown} { + screens {app, settingsController.GetAppMenu(), CreateScreenList(), Screens::ScreenListModes::UpDown} { } ApplicationList::~ApplicationList() { @@ -34,43 +37,12 @@ bool ApplicationList::OnTouchEvent(Pinetime::Applications::TouchEvents event) { return screens.OnTouchEvent(event); } -std::unique_ptr ApplicationList::CreateScreen1() { - std::array applications {{ - {Symbols::music, Apps::Music}, - {Symbols::shoe, Apps::Steps}, - {Symbols::stopWatch, Apps::StopWatch}, - {Symbols::heartBeat, Apps::HeartRate}, - {Symbols::hourGlass, Apps::Timer}, - {Symbols::alarmClock, Apps::Alarm}, - }}; - - return std::make_unique(0, 3, app, settingsController, batteryController, dateTimeController, applications); -} - -std::unique_ptr ApplicationList::CreateScreen2() { - std::array applications {{ - {Symbols::drum, Apps::Jumpscore}, - {Symbols::map, Apps::Navigation}, - {Symbols::drum, Apps::Metronome}, - {Symbols::paintbrush, Apps::Paint}, - {Symbols::paddle, Apps::Paddle}, - {"2", Apps::Twos}, - }}; +std::unique_ptr ApplicationList::CreateScreen(unsigned int screenNum) const { + std::array apps; + for (int i = 0; i < 6; i++) { + apps[i] = applications[screenNum * 6 + i]; + } - return std::make_unique(1, 3, app, settingsController, batteryController, dateTimeController, applications); + return std::make_unique(screenNum, nScreens, app, settingsController, batteryController, dateTimeController, apps); } -std::unique_ptr ApplicationList::CreateScreen3() { - std::array applications { - {Symbols::chartLine, Apps::Motion}, - // {{"A", Apps::Meter}, - // {"B", Apps::Navigation}, - // {"C", Apps::Clock}, - // {"D", Apps::Music}, - // {"E", Apps::SysInfo}, - // {"F", Apps::Brightness} - // } - }; - - return std::make_unique(2, 3, app, settingsController, batteryController, dateTimeController, applications); -} diff --git a/src/displayapp/screens/ApplicationList.h b/src/displayapp/screens/ApplicationList.h index 27ebbac..50e7fcd 100644 --- a/src/displayapp/screens/ApplicationList.h +++ b/src/displayapp/screens/ApplicationList.h @@ -7,6 +7,8 @@ #include "components/datetime/DateTimeController.h" #include "components/settings/Settings.h" #include "components/battery/BatteryController.h" +#include "displayapp/screens/Symbols.h" +#include "displayapp/screens/Tile.h" namespace Pinetime { namespace Applications { @@ -21,14 +23,32 @@ namespace Pinetime { bool OnTouchEvent(TouchEvents event) override; private: + auto CreateScreenList() const; + std::unique_ptr CreateScreen(unsigned int screenNum) const; + Controllers::Settings& settingsController; Pinetime::Controllers::Battery& batteryController; Controllers::DateTime& dateTimeController; - ScreenList<3> screens; - std::unique_ptr CreateScreen1(); - std::unique_ptr CreateScreen2(); - std::unique_ptr CreateScreen3(); + static constexpr std::array applications {{ + {Symbols::music, Apps::Music}, + {Symbols::shoe, Apps::Steps}, + {Symbols::stopWatch, Apps::StopWatch}, + {Symbols::heartBeat, Apps::HeartRate}, + {Symbols::hourGlass, Apps::Timer}, + {Symbols::clock, Apps::Alarm}, + + {Symbols::chartLine, Apps::Jumpscore}, + {Symbols::map, Apps::Navigation}, + {Symbols::drum, Apps::Metronome}, + {Symbols::paintbrush, Apps::Paint}, + {Symbols::paddle, Apps::Paddle}, + {"2", Apps::Twos}, + + {Symbols::chartLine, Apps::Motion}, + }}; + static constexpr int nScreens = (applications.size() + 5) / 6; + ScreenList screens; }; } } diff --git a/src/displayapp/screens/settings/Settings.cpp b/src/displayapp/screens/settings/Settings.cpp index bc7efcc..98d8c22 100644 --- a/src/displayapp/screens/settings/Settings.cpp +++ b/src/displayapp/screens/settings/Settings.cpp @@ -1,32 +1,27 @@ #include "displayapp/screens/settings/Settings.h" #include #include -#include "displayapp/screens/List.h" -#include "displayapp/Apps.h" #include "displayapp/DisplayApp.h" -#include "displayapp/screens/Symbols.h" +#include "displayapp/Apps.h" using namespace Pinetime::Applications::Screens; +constexpr std::array Settings::entries; + +auto Settings::CreateScreenList() const { + std::array()>, nScreens> screens; + for (size_t i = 0; i < screens.size(); i++) { + screens[i] = [this, i]() -> std::unique_ptr { + return CreateScreen(i); + }; + } + return screens; +} + Settings::Settings(Pinetime::Applications::DisplayApp* app, Pinetime::Controllers::Settings& settingsController) : Screen(app), settingsController {settingsController}, - screens {app, - settingsController.GetSettingsMenu(), - {[this]() -> std::unique_ptr { - return CreateScreen1(); - }, - [this]() -> std::unique_ptr { - return CreateScreen2(); - }, - [this]() -> std::unique_ptr { - return CreateScreen3(); - }, - [this]() -> std::unique_ptr { - return CreateScreen4(); - }, - }, - Screens::ScreenListModes::UpDown} { + screens {app, settingsController.GetSettingsMenu(), CreateScreenList(), Screens::ScreenListModes::UpDown} { } Settings::~Settings() { @@ -37,47 +32,11 @@ bool Settings::OnTouchEvent(Pinetime::Applications::TouchEvents event) { return screens.OnTouchEvent(event); } -std::unique_ptr Settings::CreateScreen1() { - std::array applications {{ - {Symbols::sun, "Display", Apps::SettingDisplay}, - {Symbols::eye, "Wake Up", Apps::SettingWakeUp}, - {Symbols::clock, "Time format", Apps::SettingTimeFormat}, - {Symbols::home, "Watch face", Apps::SettingWatchFace}, - }}; - - return std::make_unique(0, 4, app, settingsController, applications); -} - -std::unique_ptr Settings::CreateScreen2() { - std::array applications {{ - {Symbols::shoe, "Steps", Apps::SettingSteps}, - {Symbols::clock, "Set date", Apps::SettingSetDate}, - {Symbols::clock, "Set time", Apps::SettingSetTime}, - {Symbols::batteryHalf, "Battery", Apps::BatteryInfo}}}; - - return std::make_unique(1, 4, app, settingsController, applications); -} - -std::unique_ptr Settings::CreateScreen3() { - - std::array applications {{ - {Symbols::clock, "Chimes", Apps::SettingChimes}, - {Symbols::tachometer, "Shake Calib.", Apps::SettingShakeThreshold}, - {Symbols::check, "Firmware", Apps::FirmwareValidation}, - {Symbols::bluetooth, "Bluetooth", Apps::SettingBluetooth} - }}; - - return std::make_unique(2, 4, app, settingsController, applications); -} - -std::unique_ptr Settings::CreateScreen4() { - - std::array applications {{ - {Symbols::list, "About", Apps::SysInfo}, - {Symbols::none, "None", Apps::None}, - {Symbols::none, "None", Apps::None}, - {Symbols::none, "None", Apps::None} - }}; +std::unique_ptr Settings::CreateScreen(unsigned int screenNum) const { + std::array screens; + for (int i = 0; i < 4; i++) { + screens[i] = entries[screenNum * 4 + i]; + } - return std::make_unique(3, 4, app, settingsController, applications); + return std::make_unique(screenNum, nScreens, app, settingsController, screens); } diff --git a/src/displayapp/screens/settings/Settings.h b/src/displayapp/screens/settings/Settings.h index be09007..4ef507a 100644 --- a/src/displayapp/screens/settings/Settings.h +++ b/src/displayapp/screens/settings/Settings.h @@ -3,6 +3,8 @@ #include #include "displayapp/screens/Screen.h" #include "displayapp/screens/ScreenList.h" +#include "displayapp/screens/Symbols.h" +#include "displayapp/screens/List.h" namespace Pinetime { @@ -17,14 +19,34 @@ namespace Pinetime { bool OnTouchEvent(Pinetime::Applications::TouchEvents event) override; private: - Controllers::Settings& settingsController; + auto CreateScreenList() const; + std::unique_ptr CreateScreen(unsigned int screenNum) const; - ScreenList<4> screens; + Controllers::Settings& settingsController; - std::unique_ptr CreateScreen1(); - std::unique_ptr CreateScreen2(); - std::unique_ptr CreateScreen3(); - std::unique_ptr CreateScreen4(); + static constexpr std::array entries {{ + {Symbols::sun, "Display", Apps::SettingDisplay}, + {Symbols::eye, "Wake Up", Apps::SettingWakeUp}, + {Symbols::clock, "Time format", Apps::SettingTimeFormat}, + {Symbols::home, "Watch face", Apps::SettingWatchFace}, + + {Symbols::shoe, "Steps", Apps::SettingSteps}, + {Symbols::clock, "Set date", Apps::SettingSetDate}, + {Symbols::clock, "Set time", Apps::SettingSetTime}, + {Symbols::batteryHalf, "Battery", Apps::BatteryInfo}, + + {Symbols::clock, "Chimes", Apps::SettingChimes}, + {Symbols::tachometer, "Shake Calib.", Apps::SettingShakeThreshold}, + {Symbols::check, "Firmware", Apps::FirmwareValidation}, + {Symbols::bluetooth, "Bluetooth", Apps::SettingBluetooth}, + + {Symbols::list, "About", Apps::SysInfo}, + {Symbols::none, "None", Apps::None}, + {Symbols::none, "None", Apps::None}, + {Symbols::none, "None", Apps::None}, + }}; + static constexpr int nScreens = (entries.size() + 3) / 4; + ScreenList screens; }; } } -- cgit v0.10.2