Update custom colors on display when changed elswhere

src/SuperLEDstrip.ino

@@ -74,6 +74,10 @@ bool     halloweenFlickerState = false; // helper to save current state of flick
 uint16_t halloweenFlashTimer = 1000;    // initial flash timer value
 bool     halloweenFlashState = false;   // helper to save current state of flashing; true = flashing on
 
+bool updateDisplayRed   = false;
+bool updateDisplayGreen = false;
+bool updateDisplayBlue  = false;
+
 
 /*
  * Nextion display
@@ -237,4 +241,6 @@ void loop(void)
 
   Homie.loop();
 
+  updateDisplayColors();
+
 }

src/led_custom_color.h

@@ -12,6 +12,7 @@ void updateCustomColor() {
 
 void updateCustomColorRed(uint8_t red, bool updateRGB = true) {
     customColor.red = red;
+    updateDisplayRed = true;
 
     // Send updated RGB values to mqtt and display.
     // But only do so if you do not want to change other colors at the same time.
@@ -23,6 +24,7 @@ void updateCustomColorRed(uint8_t red, bool updateRGB = true) {
 
 void updateCustomColorGreen(uint8_t green, bool updateRGB = true) {
     customColor.green = green;
+    updateDisplayGreen = true;
 
     // Send updated RGB values to mqtt and display.
     // But only do so if you do not want to change other colors at the same time.
@@ -34,6 +36,7 @@ void updateCustomColorGreen(uint8_t green, bool updateRGB = true) {
 
 void updateCustomColorBlue(uint8_t blue, bool updateRGB = true) {
     customColor.blue = blue;
+    updateDisplayBlue = true;
 
     // Send updated RGB values to mqtt and display.
     // But only do so if you do not want to change other colors at the same time.

src/nextion_declaration.h

@@ -206,6 +206,6 @@ NexButton p09greenM      = NexButton(8, 23, "p09greenM");
 NexButton p09greenP      = NexButton(8, 24, "p09greenP");
 NexButton p09blueM       = NexButton(8, 26, "p09blueM");
 NexButton p09blueP       = NexButton(8, 27, "p09blueP");
-NexText   red            = NexText(  8, 18, "red");
-NexText   green          = NexText(  8, 19, "green");
-NexText   blue           = NexText(  8, 20, "blue");
+NexText   p09red         = NexText(  8, 15, "p09red");
+NexText   p09green       = NexText(  8, 22, "p09green");
+NexText   p09blue        = NexText(  8, 25, "p09blue");

src/nextion_functions.h

@@ -1,4 +1,3 @@
-
 void setTextTitle()
 {
     title.setText(DISPLAY_TITLE);
@@ -110,4 +109,63 @@ void getBlue ()
   if ( blue != 999) {
     updateCustomColorBlue(blue);
   }
+}
+
+void setRed ()
+{
+    uint8_t red = customColor.red;
+
+    // set slider
+    delay (10);     // a short delay to make the setValue work more stable
+    p09sliderRed.setValue(red);
+    delay (10);
+
+    // set value next to slider; needs to be converted to char
+    char sRed[4];
+    dtostrf(red, 3, 0, sRed);
+    p09red.setText(sRed);
+
+    updateDisplayRed = false;
+}
+
+void setGreen ()
+{
+    uint8_t green = customColor.green;
+
+    // set slider
+    delay (10);     // a short delay to make the setValue work more stable
+    p09sliderGreen.setValue(green);
+    delay (10);
+
+    // set value next to slider; needs to be converted to char
+    char sGreen[4];
+    dtostrf(green, 3, 0, sGreen);
+    p09green.setText(sGreen);
+
+    updateDisplayGreen = false;
+}
+
+void setBlue ()
+{
+    uint8_t blue = customColor.blue;
+
+    // set slider
+    delay (10);     // a short delay to make the setValue work more stable
+    p09sliderBlue.setValue(blue);
+    delay (10);
+
+    // set value next to slider; needs to be converted to char
+    char sBlue[4];
+    dtostrf(blue, 3, 0, sBlue);
+    p09blue.setText(sBlue);
+
+    updateDisplayBlue= false;
+}
+
+// this function is used because directly `setValue` in updateCustomColor* causes the program to crash
+void updateDisplayColors ()
+{
+    if ( updateDisplayRed   == true ) { setRed();   delay(10);}
+    if ( updateDisplayGreen == true ) { setGreen(); delay(10);}
+    if ( updateDisplayBlue  == true ) { setBlue(); }
 }

src/nextion_registration.h

@@ -44,9 +44,9 @@ NexTouch *nex_listen_list[] =
 
     &p09, &p09title, &p09temp, &p09humid,
     &p09settings, &p09power,
-    &red,   &p09sliderRed,   &p09redM,   &p09redP,
-    &green, &p09sliderGreen, &p09greenM, &p09greenP,
-    &blue,  &p09sliderBlue,  &p09blueM,  &p09blueP,
+    &p09red,   &p09sliderRed,   &p09redM,   &p09redP,
+    &p09green, &p09sliderGreen, &p09greenM, &p09greenP,
+    &p09blue,  &p09sliderBlue,  &p09blueM,  &p09blueP,
     NULL
 };