Working on making a brightness slider that can be reused easily

piping around IDS to correct places

Platformio/OmoteUI/UIs/BasicRefactored/page/DisplaySettings.cpp

@@ -1,26 +1,20 @@
+#include "BrightnessSlider.hpp"
 #include "DisplaySettings.hpp"
-#include "Slider.hpp"
 
 using namespace UI::Page;
 
 DisplaySettings::DisplaySettings(std::shared_ptr<DisplayAbstract> aDisplay)
     : Base(UI::ID::Pages::DisplaySettings), mDisplay(aDisplay),
-      mBrightnessSlider(
+      mBrightnessSlider(AddElement<Widget::BrightnessSlider>(std::make_unique<Widget::BrightnessSlider>(mDisplay))) {
-          AddElement<Widget::Slider>(std::make_unique<Widget::Slider>(
-              [this](auto aNewBrightness) {
-                mDisplay->setBrightness(aNewBrightness);
-              },
-              0, 255))) {
   SetBgColor(Color::GREY);
-  auto usableWidth = GetContentWidth();
 
-  mBrightnessSlider->SetWidth(usableWidth - (usableWidth * 0.20f));
+  mBrightnessSlider->SetWidth(GetContentWidth());
-  mBrightnessSlider->SetHeight(lv_pct(10));
+  mBrightnessSlider->SetHeight(50);
   mBrightnessSlider->AlignTo(this, LV_ALIGN_TOP_MID);
 }
 
 void DisplaySettings::OnShow() {
-  mBrightnessSlider->SetValue(mDisplay->getBrightness());
+
 };
 
 void DisplaySettings::OnHide(){

Platformio/OmoteUI/UIs/BasicRefactored/page/DisplaySettings.hpp

@@ -3,7 +3,7 @@
 #include "PageBase.hpp"
 
 namespace UI::Widget {
-class Slider;
+class BrightnessSlider;
 }
 
 namespace UI::Page {
@@ -18,6 +18,6 @@ public:
 
 private:
   std::shared_ptr<DisplayAbstract> mDisplay;
-  Widget::Slider *mBrightnessSlider;
+  Widget::BrightnessSlider *mBrightnessSlider;
 };
 } // namespace UI::Page

Platformio/OmoteUI/UIs/BasicRefactored/page/SettingsPage.cpp

@@ -1,6 +1,7 @@
 #include "SettingsPage.hpp"
 #include "BackgroundScreen.hpp"
 #include "Button.hpp"
+#include "Slider.hpp"
 #include "Colors.hpp"
 #include "DisplaySettings.hpp"
 #include "PopUpScreen.hpp"
@@ -30,8 +31,7 @@ void SettingsPage::PushDisplaySettings() {
 }
 
 void SettingsPage::AddSlider() {
-  auto fakeSlider = std::make_unique<UI::Widget::Base>(
+  auto fakeSlider = std::make_unique<Widget::Slider>([](auto data){});
-      lv_slider_create(UI::Screen::BackgroundScreen::getLvInstance()));
   fakeSlider->SetHeight(lv_pct(10));
   fakeSlider->SetWidth(GetContentWidth());
   if (sliders.empty()) {

Platformio/OmoteUI/UIs/BasicRefactored/widget/BrightnessSlider.cpp

@@ -0,0 +1,33 @@
+#include "BrightnessSlider.hpp"
+#include "Label.hpp"
+#include "Slider.hpp"
+
+using namespace UI::Widget;
+
+
+BrightnessSlider::BrightnessSlider(std::shared_ptr<DisplayAbstract> aDisplay): Base(ID::Widgets::BrightnessSlider),
+  mDisplay(aDisplay),
+  mSlider(AddElement<Widget::Slider>(std::make_unique<Slider>(
+    [this](auto aNewBrightness){
+        mDisplay->setBrightness(aNewBrightness);
+    },0,255))),
+  mLabel(AddElement<Widget::Label>(std::make_unique<Label>("Brightness")))
+{
+    mLabel->AlignTo(this,LV_ALIGN_TOP_MID);
+    mSlider->AlignTo(mLabel,LV_ALIGN_OUT_BOTTOM_MID);
+    mSlider->SetWidth(GetContentWidth() - 20);
+}
+
+void BrightnessSlider::OnShow(){
+    mSlider->SetValue(mDisplay->getBrightness());
+}
+
+void BrightnessSlider::SetHeight(lv_coord_t aHeight){
+    Base::SetHeight(aHeight);
+    auto labelHeight = GetContentHeight() * 0.25f;
+    auto sliderHeight = aHeight - labelHeight;
+    mLabel->SetHeight(labelHeight);
+    mSlider->SetHeight(sliderHeight);
+    mLabel->AlignTo(this,LV_ALIGN_TOP_MID);
+    mSlider->AlignTo(mLabel,LV_ALIGN_OUT_BOTTOM_MID);
+}

Platformio/OmoteUI/UIs/BasicRefactored/widget/BrightnessSlider.hpp

@@ -0,0 +1,21 @@
+#pragma once
+#include "WidgetBase.hpp"
+#include "DisplayAbstract.h"
+
+namespace UI::Widget{
+class Slider;
+class Label;
+
+class BrightnessSlider : public Base{
+public:
+    BrightnessSlider(std::shared_ptr<DisplayAbstract> aDisplay);
+
+    void OnShow() override;
+    void SetHeight(lv_coord_t aHeight) override;
+private:
+    std::shared_ptr<DisplayAbstract> mDisplay;
+    Slider* mSlider;
+    Label* mLabel;
+};
+
+}

Platformio/OmoteUI/core/UIElementIds.hpp

@@ -15,6 +15,9 @@ public:
 
   enum class Widgets {
     Slider = static_cast<int>(Screens::INVALID_SCREEN_ID) + 1,
+    Button,
+    Label,
+    BrightnessSlider,
     INVALID_WIDGET_ID
   };
 

Platformio/OmoteUI/core/widget/Button.cpp

@@ -4,7 +4,7 @@
 using namespace UI::Widget;
 
 Button::Button(std::function<void()> aOnPressHandler)
-    : Base(lv_btn_create(UI::Screen::BackgroundScreen::getLvInstance())),
+    : Base(lv_btn_create(UI::Screen::BackgroundScreen::getLvInstance()),ID::Widgets::Button),
       mOnPress(aOnPressHandler) {}
 
 void Button::OnLvglEvent(lv_event_t *anEvent) {

Platformio/OmoteUI/core/widget/Label.cpp

@@ -5,6 +5,6 @@
 using namespace UI::Widget;
 
 Label::Label(std::string aText)
-    : Base(lv_label_create(UI::Screen::BackgroundScreen::getLvInstance())) {
+    : Base(lv_label_create(UI::Screen::BackgroundScreen::getLvInstance()), ID::Widgets::Label) {
   lv_label_set_text(LvglSelf(), aText.c_str());
 }

Platformio/OmoteUI/core/widget/Slider.cpp

@@ -6,7 +6,7 @@ using namespace UI::Widget;
 
 Slider::Slider(std::function<void(int32_t)> aOnSliderValueChange,
                int32_t aMinVal, int32_t aMaxVal)
-    : Base(lv_slider_create(UI::Screen::BackgroundScreen::getLvInstance())),
+    : Base(lv_slider_create(UI::Screen::BackgroundScreen::getLvInstance()),ID::Widgets::Slider),
       mOnSliderChange(std::move(aOnSliderValueChange)) {
   auto lock = LvglResourceManager::GetInstance().scopeLock();
   lv_slider_set_range(LvglSelf(), aMinVal, aMaxVal);

Platformio/OmoteUI/core/widget/WidgetBase.cpp

@@ -1,8 +1,15 @@
 #include "WidgetBase.hpp"
+#include "BackgroundScreen.hpp"
 
+using namespace UI;
 using namespace UI::Widget;
 
-Base::Base(lv_obj_t *aLvglSelf) : UIElement(aLvglSelf) {
+Base::Base(ID anId): UIElement(lv_obj_create(Screen::BackgroundScreen::getLvInstance()),anId){
+  SetWidth(lv_pct(100));
+  SetHeight(lv_pct(100));
+}
+
+Base::Base(lv_obj_t *aLvglSelf, ID anId) : UIElement(aLvglSelf,anId) {
   SetWidth(lv_pct(100));
   SetHeight(lv_pct(100));
 }

Platformio/OmoteUI/core/widget/WidgetBase.hpp

@@ -19,7 +19,8 @@ class Base : public UIElement {
 public:
   typedef std::unique_ptr<Base> Ptr;
 
-  Base(lv_obj_t *aLvglSelf);
+  Base(ID anId);
+  Base(lv_obj_t *aLvglSelf, ID anId);
   virtual ~Base() override = default;
 
 protected: