diff --git a/components/esp-gui-lib b/components/esp-gui-lib index a279559..ec002d6 160000 --- a/components/esp-gui-lib +++ b/components/esp-gui-lib @@ -1 +1 @@ -Subproject commit a279559e768b70c919a1516f8401a4fae4821c54 +Subproject commit ec002d60587626080f593b8551bfef582d9308f5 diff --git a/main/changevaluedisplay_controlmode.cpp b/main/changevaluedisplay_controlmode.cpp index e69de29..d2fb5c1 100644 --- a/main/changevaluedisplay_controlmode.cpp +++ b/main/changevaluedisplay_controlmode.cpp @@ -0,0 +1,53 @@ +#include "changevaluedisplay_controlmode.h" + +// esp-idf includes +#include + +// 3rdparty lib includes +#include +#include +#include +#include + +// local includes +#include "utils.h" + +namespace espgui { +namespace { +constexpr const char * const TAG = "ESPGUI"; + +constexpr char TEXT_OPENMODE[] = "Open mode"; +constexpr char TEXT_VOLTAGE[] = "Voltage"; +constexpr char TEXT_SPEED[] = "Speed"; +constexpr char TEXT_TORQUE[] = "Torque"; +constexpr char TEXT_BACK[] = "Back"; +} // namespace + +ChangeValueDisplay::ChangeValueDisplay() +{ + using bobbycar::protocol::ControlMode; + constructMenuItem, StaticText>>(ControlMode::OpenMode, *this, *this, *this); + constructMenuItem, StaticText>>(ControlMode::Voltage, *this, *this, *this); + constructMenuItem, StaticText>>(ControlMode::Speed, *this, *this, *this); + constructMenuItem, StaticText>>(ControlMode::Torque, *this, *this, *this); + constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); +} + +void ChangeValueDisplay::start() +{ + Base::start(); + + switch (const auto value = getValue()) + { + using bobbycar::protocol::ControlMode; + case ControlMode::OpenMode: setSelectedIndex(0); break; + case ControlMode::Voltage: setSelectedIndex(1); break; + case ControlMode::Speed: setSelectedIndex(2); break; + case ControlMode::Torque: setSelectedIndex(3); break; + default: + ESP_LOGW(TAG, "Unknown ControlMode: %i", std::to_underlying(value)); + setSelectedIndex(4); + } +} + +} // namespace espgui diff --git a/main/changevaluedisplay_controlmode.h b/main/changevaluedisplay_controlmode.h index 91abbf2..566f1fb 100644 --- a/main/changevaluedisplay_controlmode.h +++ b/main/changevaluedisplay_controlmode.h @@ -1,23 +1,22 @@ #pragma once -// esp-idf includes -#include +// 3rdparty lib includes +#include +#include +#include +#include // local includes -#include "changevaluedisplay.h" -#include "menudisplay.h" -#include "utils.h" -#include "actions/setvalueaction.h" -#include "actions/backproxyaction.h" -#include "icons/back.h" -#include "texts.h" +#include "bobbycar-common.h" namespace espgui { + template<> class ChangeValueDisplay : public MenuDisplay, public virtual AccessorInterface, - public virtual ActionInterface + public virtual ConfirmInterface, + public virtual ErrorHandlerInterface { using Base = MenuDisplay; @@ -27,30 +26,4 @@ public: void start() override; }; -ChangeValueDisplay::ChangeValueDisplay() -{ - using bobbycar::protocol::ControlMode; - constructMenuItem, StaticText>>(ControlMode::OpenMode, *this, *this); - constructMenuItem, StaticText>>(ControlMode::Voltage, *this, *this); - constructMenuItem, StaticText>>(ControlMode::Speed, *this, *this); - constructMenuItem, StaticText>>(ControlMode::Torque, *this, *this); - constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); -} - -void ChangeValueDisplay::start() -{ - Base::start(); - - switch (const auto value = getValue()) - { - using bobbycar::protocol::ControlMode; - case ControlMode::OpenMode: setSelectedIndex(0); break; - case ControlMode::Voltage: setSelectedIndex(1); break; - case ControlMode::Speed: setSelectedIndex(2); break; - case ControlMode::Torque: setSelectedIndex(3); break; - default: - ESP_LOGW("BOBBY", "Unknown ControlMode: %i", int(value)); - setSelectedIndex(4); - } -} } // namespace espgui diff --git a/main/changevaluedisplay_controltype.cpp b/main/changevaluedisplay_controltype.cpp index e69de29..7363120 100644 --- a/main/changevaluedisplay_controltype.cpp +++ b/main/changevaluedisplay_controltype.cpp @@ -0,0 +1,50 @@ +#include "changevaluedisplay_controltype.h" + +// esp-idf includes +#include + +// 3rdparty lib includes +#include +#include +#include +#include + +// local includes +#include "utils.h" + +namespace espgui { +namespace { +constexpr const char * const TAG = "ESPGUI"; + +constexpr char TEXT_COMMUTATION[] = "Commutation"; +constexpr char TEXT_SINUSOIDAL[] = "Sinusoidal"; +constexpr char TEXT_FIELDORIENTEDCONTROL[] = "Field oriented control"; +constexpr char TEXT_BACK[] = "Back"; +} // namespace + +ChangeValueDisplay::ChangeValueDisplay() +{ + using bobbycar::protocol::ControlType; + constructMenuItem, StaticText>>(ControlType::Commutation, *this, *this, *this); + constructMenuItem, StaticText>>(ControlType::Sinusoidal, *this, *this, *this); + constructMenuItem, StaticText>>(ControlType::FieldOrientedControl, *this, *this, *this); + constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); +} + +void ChangeValueDisplay::start() +{ + Base::start(); + + switch (const auto value = getValue()) + { + using bobbycar::protocol::ControlType; + case ControlType::Commutation: setSelectedIndex(0); break; + case ControlType::Sinusoidal: setSelectedIndex(1); break; + case ControlType::FieldOrientedControl: setSelectedIndex(2); break; + default: + ESP_LOGW(TAG, "Unknown ControlType: %i", std::to_underlying(value)); + setSelectedIndex(3); + } +} + +} // namespace espgui diff --git a/main/changevaluedisplay_controltype.h b/main/changevaluedisplay_controltype.h index fa84ba0..10d9ab5 100644 --- a/main/changevaluedisplay_controltype.h +++ b/main/changevaluedisplay_controltype.h @@ -1,23 +1,22 @@ #pragma once -// esp-idf includes -#include +// 3rdparty lib includes +#include +#include +#include +#include // local includes -#include "changevaluedisplay.h" -#include "menudisplay.h" -#include "utils.h" -#include "actions/setvalueaction.h" -#include "actions/backproxyaction.h" -#include "icons/back.h" -#include "texts.h" +#include "bobbycar-common.h" namespace espgui { + template<> class ChangeValueDisplay : public MenuDisplay, public virtual AccessorInterface, - public virtual ActionInterface + public virtual ConfirmInterface, + public virtual ErrorHandlerInterface { using Base = MenuDisplay; @@ -27,28 +26,4 @@ public: void start() override; }; -ChangeValueDisplay::ChangeValueDisplay() -{ - using bobbycar::protocol::ControlType; - constructMenuItem, StaticText>>(ControlType::Commutation, *this, *this); - constructMenuItem, StaticText>>(ControlType::Sinusoidal, *this, *this); - constructMenuItem, StaticText>>(ControlType::FieldOrientedControl, *this, *this); - constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); -} - -void ChangeValueDisplay::start() -{ - Base::start(); - - switch (const auto value = getValue()) - { - using bobbycar::protocol::ControlType; - case ControlType::Commutation: setSelectedIndex(0); break; - case ControlType::Sinusoidal: setSelectedIndex(1); break; - case ControlType::FieldOrientedControl: setSelectedIndex(2); break; - default: - ESP_LOGW("BOBBY", "Unknown ControlType: %i", int(value)); - setSelectedIndex(3); - } -} } // namespace espgui diff --git a/main/changevaluedisplay_handbremsmode.cpp b/main/changevaluedisplay_handbremsmode.cpp index 80135a3..e5574d8 100644 --- a/main/changevaluedisplay_handbremsmode.cpp +++ b/main/changevaluedisplay_handbremsmode.cpp @@ -3,16 +3,31 @@ // esp-idf includes #include +// 3rdparty lib includes +#include +#include +#include +#include + // local includes #include "utils.h" -#include "texts.h" namespace espgui { +namespace { +constexpr const char * const TAG = "ESPGUI"; + +constexpr char TEXT_HANDBREMS_MOSFETS_OFF[] = "Mosfets off"; +constexpr char TEXT_HANDBREMS_OPENMODE[] = "Open Mode"; +constexpr char TEXT_HANDBREMS_SPEED0[] = "Speed 0"; +constexpr char TEXT_HANDBREMSE[] = "Handbremse"; +constexpr char TEXT_BACK[] = "Back"; +} // namespace + ChangeValueDisplay::ChangeValueDisplay() { - constructMenuItem, StaticText>>(HandbremseMode::MOSFETS_OFF, *this, *this); - constructMenuItem, StaticText>>(HandbremseMode::OPENMODE, *this, *this); - constructMenuItem, StaticText>>(HandbremseMode::SPEED_0, *this, *this); + constructMenuItem, StaticText>>(HandbremseMode::MOSFETS_OFF, *this, *this, *this); + constructMenuItem, StaticText>>(HandbremseMode::OPENMODE, *this, *this, *this); + constructMenuItem, StaticText>>(HandbremseMode::SPEED_0, *this, *this, *this); constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); } @@ -26,8 +41,9 @@ void ChangeValueDisplay::start() case HandbremseMode::OPENMODE: setSelectedIndex(1); break; case HandbremseMode::SPEED_0: setSelectedIndex(2); break; default: - ESP_LOGW("BOBBY", "Unknown HandbremseMode: %i", int(value)); + ESP_LOGW(TAG, "Unknown HandbremseMode: %i", std::to_underlying(value)); setSelectedIndex(3); } } + } // namespace espgui diff --git a/main/changevaluedisplay_handbremsmode.h b/main/changevaluedisplay_handbremsmode.h index fe156f4..800739f 100644 --- a/main/changevaluedisplay_handbremsmode.h +++ b/main/changevaluedisplay_handbremsmode.h @@ -1,21 +1,22 @@ #pragma once // 3rdparty lib includes -#include "changevaluedisplay.h" -#include "menudisplay.h" -#include "actions/setvalueaction.h" -#include "actions/backproxyaction.h" -#include "icons/back.h" +#include +#include +#include +#include // local includes #include "handbremse.h" namespace espgui { + template<> class ChangeValueDisplay : public MenuDisplay, public virtual AccessorInterface, - public virtual ActionInterface + public virtual ConfirmInterface, + public virtual ErrorHandlerInterface { using Base = MenuDisplay; @@ -24,4 +25,5 @@ public: void start() override; }; + } // namespace espgui diff --git a/main/changevaluedisplay_larsmmode_mode.cpp b/main/changevaluedisplay_larsmmode_mode.cpp index e69de29..20469ae 100644 --- a/main/changevaluedisplay_larsmmode_mode.cpp +++ b/main/changevaluedisplay_larsmmode_mode.cpp @@ -0,0 +1,51 @@ +#include "changevaluedisplay_larsmmode_mode.h" + +// esp-idf includes +#include + +// 3rdparty lib inclues +#include +#include +#include +#include + +// local includes +#include "utils.h" + +namespace espgui { +namespace { +constexpr const char * const TAG = "ESPGUI"; + +constexpr char TEXT_LARSMMODE1[] = "Mode1"; +constexpr char TEXT_LARSMMODE2[] = "Mode2"; +constexpr char TEXT_LARSMMODE3[] = "Mode3"; +constexpr char TEXT_LARSMMODE4[] = "Mode4"; +constexpr char TEXT_BACK[] = "Back"; +} // namespace + +ChangeValueDisplay::ChangeValueDisplay() +{ + constructMenuItem, StaticText>>(LarsmModeMode::Mode1, *this, *this, *this); + constructMenuItem, StaticText>>(LarsmModeMode::Mode2, *this, *this, *this); + constructMenuItem, StaticText>>(LarsmModeMode::Mode3, *this, *this, *this); + constructMenuItem, StaticText>>(LarsmModeMode::Mode4, *this, *this, *this); + constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); +} + +void ChangeValueDisplay::start() +{ + Base::start(); + + switch (const auto value = getValue()) + { + case LarsmModeMode::Mode1: setSelectedIndex(0); break; + case LarsmModeMode::Mode2: setSelectedIndex(1); break; + case LarsmModeMode::Mode3: setSelectedIndex(2); break; + case LarsmModeMode::Mode4: setSelectedIndex(3); break; + default: + ESP_LOGW(TAG, "Unknown LarsmModeMode: %i", std::to_underlying(value)); + setSelectedIndex(4); + } +} + +} // namespace espgui diff --git a/main/changevaluedisplay_larsmmode_mode.h b/main/changevaluedisplay_larsmmode_mode.h index d9d1834..64931a5 100644 --- a/main/changevaluedisplay_larsmmode_mode.h +++ b/main/changevaluedisplay_larsmmode_mode.h @@ -1,22 +1,22 @@ #pragma once -#include +// 3rdparty lib inclues +#include +#include +#include +#include -#include "changevaluedisplay.h" -#include "menudisplay.h" -#include "utils.h" -#include "actions/setvalueaction.h" -#include "actions/backproxyaction.h" -#include "icons/back.h" -#include "texts.h" +// local includes #include "modes/larsmmode.h" namespace espgui { + template<> class ChangeValueDisplay : public MenuDisplay, public virtual AccessorInterface, - public virtual ActionInterface + public virtual ConfirmInterface, + public virtual ErrorHandlerInterface { using Base = MenuDisplay; @@ -26,28 +26,4 @@ public: void start() override; }; -ChangeValueDisplay::ChangeValueDisplay() -{ - constructMenuItem, StaticText>>(LarsmModeMode::Mode1, *this, *this); - constructMenuItem, StaticText>>(LarsmModeMode::Mode2, *this, *this); - constructMenuItem, StaticText>>(LarsmModeMode::Mode3, *this, *this); - constructMenuItem, StaticText>>(LarsmModeMode::Mode4, *this, *this); - constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); -} - -void ChangeValueDisplay::start() -{ - Base::start(); - - switch (const auto value = getValue()) - { - case LarsmModeMode::Mode1: setSelectedIndex(0); break; - case LarsmModeMode::Mode2: setSelectedIndex(1); break; - case LarsmModeMode::Mode3: setSelectedIndex(2); break; - case LarsmModeMode::Mode4: setSelectedIndex(3); break; - default: - ESP_LOGW("BOBBY", "Unknown LarsmModeMode: %i", int(value)); - setSelectedIndex(4); - } -} } // namespace espgui diff --git a/main/changevaluedisplay_unifiedmodelmode.cpp b/main/changevaluedisplay_unifiedmodelmode.cpp index 9f3ab55..0aa7e43 100644 --- a/main/changevaluedisplay_unifiedmodelmode.cpp +++ b/main/changevaluedisplay_unifiedmodelmode.cpp @@ -3,18 +3,34 @@ // esp-idf includes #include +// 3rdparty lib includes +#include +#include +#include +#include + // local includes #include "utils.h" -#include "texts.h" namespace espgui { +namespace { +constexpr const char * const TAG = "ESPGUI"; + +constexpr char TEXT_COMMUTATION[] = "Commutation"; +constexpr char TEXT_SINUSOIDAL[] = "Sinusoidal"; +constexpr char TEXT_FOCVOLTAGE[] = "FOC/Voltage"; +constexpr char TEXT_FOCSPEED[] = "FOC/Speed"; +constexpr char TEXT_FOCTORQUE[] = "FOC/Torque"; +constexpr char TEXT_BACK[] = "Back"; +} // namespace + ChangeValueDisplay::ChangeValueDisplay() { - constructMenuItem, StaticText>>(UnifiedModelMode::Commutation, *this, *this); - constructMenuItem, StaticText>>(UnifiedModelMode::Sinusoidal, *this, *this); - constructMenuItem, StaticText>>(UnifiedModelMode::FocVoltage, *this, *this); - constructMenuItem, StaticText>>(UnifiedModelMode::FocSpeed, *this, *this); - constructMenuItem, StaticText>>(UnifiedModelMode::FocTorque, *this, *this); + constructMenuItem, StaticText>>(UnifiedModelMode::Commutation, *this, *this, *this); + constructMenuItem, StaticText>>(UnifiedModelMode::Sinusoidal, *this, *this, *this); + constructMenuItem, StaticText>>(UnifiedModelMode::FocVoltage, *this, *this, *this); + constructMenuItem, StaticText>>(UnifiedModelMode::FocSpeed, *this, *this, *this); + constructMenuItem, StaticText>>(UnifiedModelMode::FocTorque, *this, *this, *this); constructMenuItem, StaticMenuItemIcon<&espgui::icons::back>>>(*this); } @@ -30,8 +46,9 @@ void ChangeValueDisplay::start() case UnifiedModelMode::FocSpeed: setSelectedIndex(3); break; case UnifiedModelMode::FocTorque: setSelectedIndex(4); break; default: - ESP_LOGW("BOBBY", "Unknown UnifiedModelMode: %i", int(value)); + ESP_LOGW(TAG, "Unknown UnifiedModelMode: %i", std::to_underlying(value)); setSelectedIndex(5); } } + } // namespace espgui diff --git a/main/changevaluedisplay_unifiedmodelmode.h b/main/changevaluedisplay_unifiedmodelmode.h index 3e7b120..e6f4bbe 100644 --- a/main/changevaluedisplay_unifiedmodelmode.h +++ b/main/changevaluedisplay_unifiedmodelmode.h @@ -1,21 +1,22 @@ #pragma once // 3rdparty lib includes -#include "changevaluedisplay.h" -#include "menudisplay.h" -#include "actions/setvalueaction.h" -#include "actions/backproxyaction.h" -#include "icons/back.h" +#include +#include +#include +#include // local includes #include "unifiedmodelmode.h" namespace espgui { + template<> class ChangeValueDisplay : public MenuDisplay, public virtual AccessorInterface, - public virtual ActionInterface + public virtual ConfirmInterface, + public virtual ErrorHandlerInterface { using Base = MenuDisplay; @@ -24,4 +25,5 @@ public: void start() override; }; + } // namespace espgui diff --git a/main/displays/bobbypopupdisplay.cpp b/main/displays/bobbypopupdisplay.cpp index d163303..5a3cf38 100644 --- a/main/displays/bobbypopupdisplay.cpp +++ b/main/displays/bobbypopupdisplay.cpp @@ -19,7 +19,7 @@ void BobbyPopupDisplay::rawButtonReleased(uint8_t button) void BobbyPopupDisplay::buttonPressed(espgui::Button button) { - //Base::buttonPressed(button); + Base::buttonPressed(button); buttonPressedCommon(button); } diff --git a/main/displays/menus/defaultmodesettingsmenu.cpp b/main/displays/menus/defaultmodesettingsmenu.cpp index d62a0e9..6d32adc 100644 --- a/main/displays/menus/defaultmodesettingsmenu.cpp +++ b/main/displays/menus/defaultmodesettingsmenu.cpp @@ -23,15 +23,15 @@ using DefaultModeModelModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, DefaultModeModelModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; using DefaultModeHybridModelModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, DefaultModeHybridModelModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; using DefaultModeSmoothingChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, @@ -126,8 +126,8 @@ DefaultModeSettingsMenu::DefaultModeSettingsMenu() constructMenuItem, ToggleBoolAction, CheckboxIcon, DefaultModeEnableSmoothingDownAccessor>>(); constructMenuItem, ToggleBoolAction, CheckboxIcon, DefaultModeEnableFieldWeakSmoothingUpAccessor>>(); constructMenuItem, ToggleBoolAction, CheckboxIcon, DefaultModeEnableFieldWeakSmoothingDownAccessor>>(); - constructMenuItem, ToggleBoolAction, CheckboxIcon, DefaultModeEnableHybridAccessor>>(); - constructMenuItem, SwitchScreenAction>>(); + constructMenuItem, ToggleBoolAction, CheckboxIcon, DefaultModeEnableHybridAccessor>>(); + constructMenuItem, SwitchScreenAction>>(); constructMenuItem, SwitchScreenAction>>(); constructMenuItem, SwitchScreenAction>>(); constructMenuItem, SwitchScreenAction>>(); diff --git a/main/displays/menus/handbremssettingsmenu.cpp b/main/displays/menus/handbremssettingsmenu.cpp index 182e1c4..118eaf2 100644 --- a/main/displays/menus/handbremssettingsmenu.cpp +++ b/main/displays/menus/handbremssettingsmenu.cpp @@ -28,8 +28,8 @@ using HandBremsModeChangeValueDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, HandbremsModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; class HandBremsModeText : public virtual espgui::TextInterface { diff --git a/main/displays/menus/handbremssettingsmenu.h b/main/displays/menus/handbremssettingsmenu.h index aeffeb9..3ade5ec 100644 --- a/main/displays/menus/handbremssettingsmenu.h +++ b/main/displays/menus/handbremssettingsmenu.h @@ -6,7 +6,7 @@ class HandbremsSettingsMenu : public BobbyMenuDisplay, - public espgui::StaticText + public espgui::StaticText { public: HandbremsSettingsMenu(); diff --git a/main/displays/menus/larsmmodesettingsmenu.cpp b/main/displays/menus/larsmmodesettingsmenu.cpp index 66c2cbf..010cdaa 100644 --- a/main/displays/menus/larsmmodesettingsmenu.cpp +++ b/main/displays/menus/larsmmodesettingsmenu.cpp @@ -19,15 +19,15 @@ using LarsmModeModelModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, LarsmModeModelModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; using LarsmModeModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, LarsmModeModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; using LarsmModeIterationsChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, diff --git a/main/displays/menus/tempomatmodesettingsmenu.cpp b/main/displays/menus/tempomatmodesettingsmenu.cpp index f83956d..441f2cc 100644 --- a/main/displays/menus/tempomatmodesettingsmenu.cpp +++ b/main/displays/menus/tempomatmodesettingsmenu.cpp @@ -29,8 +29,8 @@ using TempomatModeModelModeChangeScreen = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, TempomatModeModelModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; } // namespace diff --git a/main/displays/menus/timesettingsmenu.cpp b/main/displays/menus/timesettingsmenu.cpp index 5ba1e7e..5f22529 100644 --- a/main/displays/menus/timesettingsmenu.cpp +++ b/main/displays/menus/timesettingsmenu.cpp @@ -56,8 +56,8 @@ using DaylightSavingModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, DaylightSavingModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; #ifdef FEATURE_NTP @@ -65,8 +65,8 @@ using TimeSyncModeChangeDisplay = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, TimeSyncModeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; using TimeSyncIntervalChangeDisplay = espgui::makeComponent< diff --git a/main/displays/menus/wifiapsettingsmenu.cpp b/main/displays/menus/wifiapsettingsmenu.cpp index e0f2789..ecf91ff 100644 --- a/main/displays/menus/wifiapsettingsmenu.cpp +++ b/main/displays/menus/wifiapsettingsmenu.cpp @@ -63,8 +63,8 @@ using ApAuthmodeChangeScreen = espgui::makeComponent< BobbyChangeValueDisplay, espgui::StaticText, WifiApAuthmodeAccessor, - espgui::BackActionInterface>, - espgui::SwitchScreenAction + espgui::ConfirmActionInterface>, + espgui::BackActionInterface> >; } // namespace diff --git a/main/modes/larsmmode.h b/main/modes/larsmmode.h index 08e28c0..873f271 100644 --- a/main/modes/larsmmode.h +++ b/main/modes/larsmmode.h @@ -4,6 +4,8 @@ #include "bobbycar-common.h" #include "modeinterface.h" +enum class LarsmModeMode : uint8_t { Mode1, Mode2, Mode3, Mode4 }; + class LarsmMode : public ModeInterface { using Base = ModeInterface; diff --git a/main/settings.h b/main/settings.h index 0a16df3..c44382b 100644 --- a/main/settings.h +++ b/main/settings.h @@ -23,8 +23,7 @@ #include "unifiedmodelmode.h" #include "handbremse.h" #include "ledstrip.h" - -enum class LarsmModeMode : uint8_t { Mode1, Mode2, Mode3, Mode4 }; +#include "modes/larsmmode.h" struct Settings { diff --git a/main/texts.cpp b/main/texts.cpp index 1c75d23..4617728 100644 --- a/main/texts.cpp +++ b/main/texts.cpp @@ -234,6 +234,7 @@ char TEXT_HANDBREMSE_AUTOMATIC[] = "Automatic Handbremse"; char TEXT_HANDBREMSE_VISUALIZE[] = "Visualize Handbremse"; char TEXT_HANDBREMSE_MODE[] = "Handbrems Mode"; char TEXT_HANDBREMSE_TRIGGERTIMEOUT[] = "Handbrems Timeout"; +char TEXT_HANDBREMSE[] = "Handbremse"; //char TEXT_BACK[] = "Back"; //DynamicDebugMenu @@ -448,62 +449,6 @@ char TEXT_OFF[] = "Off"; char TEXT_MASTER[] = "Master"; char TEXT_SLAVE[] = "Slave"; -//ChangeValueDisplay -char TEXT_OPENMODE[] = "Open mode"; -char TEXT_VOLTAGE[] = "Voltage"; -char TEXT_SPEED[] = "Speed"; -char TEXT_TORQUE[] = "Torque"; -//char TEXT_BACK[] = "Back"; - -//ChangeValueDisplay -char TEXT_COMMUTATION[] = "Commutation"; -char TEXT_SINUSOIDAL[] = "Sinusoidal"; -char TEXT_FIELDORIENTEDCONTROL[] = "Field oriented control"; -//char TEXT_BACK[] = "Back"; - -//ChangeValueDisplay -char TEXT_HANDBREMS_MOSFETS_OFF[] = "Mosfets off"; -char TEXT_HANDBREMS_OPENMODE[] = "Open Mode"; -char TEXT_HANDBREMS_SPEED0[] = "Speed 0"; -char TEXT_HANDBREMSE[] = "Handbremse"; - -//ChangeValueDisplay -char TEXT_LARSMMODE1[] = "Mode1"; -char TEXT_LARSMMODE2[] = "Mode2"; -char TEXT_LARSMMODE3[] = "Mode3"; -char TEXT_LARSMMODE4[] = "Mode4"; -//char TEXT_BACK[] = "Back"; - -//ChangeValueDisplay -//char TEXT_COMMUTATION[] = "Commutation"; -//char TEXT_SINUSOIDAL[] = "Sinusoidal"; -char TEXT_FOCVOLTAGE[] = "FOC/Voltage"; -char TEXT_FOCSPEED[] = "FOC/Speed"; -char TEXT_FOCTORQUE[] = "FOC/Torque"; -//char TEXT_BACK[] = "Back"; - -//ChangeValueDisplay -char TEXT_WIFI_MODE_NULL[] = "WIFI_MODE_NULL"; -char TEXT_WIFI_MODE_STA[] = "WIFI_MODE_STA"; -char TEXT_WIFI_MODE_AP[] = "WIFI_MODE_AP"; -char TEXT_WIFI_MODE_APSTA[] = "WIFI_MODE_APSTA"; -//char TEXT_BACK[] = "Back"; - -//ChangeValueDisplay -char TEXT_WIFI_POWER_19_5dBm[] = "19.5dBm"; -char TEXT_WIFI_POWER_19dBm[] = "19dBm"; -char TEXT_WIFI_POWER_18_5dBm[] = "18.5dBm"; -char TEXT_WIFI_POWER_17dBm[] = "17dBm"; -char TEXT_WIFI_POWER_15dBm[] = "15dBm"; -char TEXT_WIFI_POWER_13dBm[] = "13dBm"; -char TEXT_WIFI_POWER_11dBm[] = "11dBm"; -char TEXT_WIFI_POWER_8_5dBm[] = "8.5dBm"; -char TEXT_WIFI_POWER_7dBm[] = "7dBm"; -char TEXT_WIFI_POWER_5dBm[] = "5dBm"; -char TEXT_WIFI_POWER_2dBm[] = "2dBm"; -char TEXT_WIFI_POWER_MINUS_1dBm[] = "-1dBm"; -//char TEXT_BACK[] = "Back"; - //Crashmenu char TEXT_CRASHMENU[] = "Crash Menu"; char TEXT_CRASH_ASSERT[] = "assert(0)"; diff --git a/main/texts.h b/main/texts.h index e9c08ec..b24807d 100644 --- a/main/texts.h +++ b/main/texts.h @@ -448,61 +448,6 @@ extern char TEXT_OFF[]; extern char TEXT_MASTER[]; extern char TEXT_SLAVE[]; -//ChangeValueDisplay -extern char TEXT_OPENMODE[]; -extern char TEXT_VOLTAGE[]; -extern char TEXT_SPEED[]; -extern char TEXT_TORQUE[]; -//extern char TEXT_BACK[]; - -//ChangeValueDisplay -extern char TEXT_COMMUTATION[]; -extern char TEXT_SINUSOIDAL[]; -extern char TEXT_FIELDORIENTEDCONTROL[]; -//extern char TEXT_BACK[]; - -//ChangeValueDisplay -extern char TEXT_HANDBREMS_MOSFETS_OFF[]; -extern char TEXT_HANDBREMS_OPENMODE[]; -extern char TEXT_HANDBREMS_SPEED0[]; - -//ChangeValueDisplay -extern char TEXT_LARSMMODE1[]; -extern char TEXT_LARSMMODE2[]; -extern char TEXT_LARSMMODE3[]; -extern char TEXT_LARSMMODE4[]; -//extern char TEXT_BACK[]; - -//ChangeValueDisplay -//extern char TEXT_COMMUTATION[]; -//extern char TEXT_SINUSOIDAL[]; -extern char TEXT_FOCVOLTAGE[]; -extern char TEXT_FOCSPEED[]; -extern char TEXT_FOCTORQUE[]; -//extern char TEXT_BACK[]; - -//ChangeValueDisplay -extern char TEXT_WIFI_MODE_NULL[]; -extern char TEXT_WIFI_MODE_STA[]; -extern char TEXT_WIFI_MODE_AP[]; -extern char TEXT_WIFI_MODE_APSTA[]; -//extern char TEXT_BACK[]; - -//ChangeValueDisplay -extern char TEXT_WIFI_POWER_19_5dBm[]; -extern char TEXT_WIFI_POWER_19dBm[]; -extern char TEXT_WIFI_POWER_18_5dBm[]; -extern char TEXT_WIFI_POWER_17dBm[]; -extern char TEXT_WIFI_POWER_15dBm[]; -extern char TEXT_WIFI_POWER_13dBm[]; -extern char TEXT_WIFI_POWER_11dBm[]; -extern char TEXT_WIFI_POWER_8_5dBm[]; -extern char TEXT_WIFI_POWER_7dBm[]; -extern char TEXT_WIFI_POWER_5dBm[]; -extern char TEXT_WIFI_POWER_2dBm[]; -extern char TEXT_WIFI_POWER_MINUS_1dBm[]; -//extern char TEXT_BACK[]; - //Crashmenu extern char TEXT_CRASHMENU[]; extern char TEXT_CRASH_ASSERT[];