From d60506a75d30a03c9c61249430f32510b59ba352 Mon Sep 17 00:00:00 2001
From: Blaz Kristan <blaz@kristan-sp.si>
Date: Wed, 24 Feb 2021 21:33:44 +0100
Subject: [PATCH] Skip 1st fix, save LED count, removed reverse.

---
 wled00/bus_manager.h          | 4 ++--
 wled00/data/settings_leds.htm | 1 -
 wled00/html_settings.h        | 5 ++---
 wled00/set.cpp                | 4 ++--
 4 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/wled00/bus_manager.h b/wled00/bus_manager.h
index 610a56bf4..41cb6a1d7 100644
--- a/wled00/bus_manager.h
+++ b/wled00/bus_manager.h
@@ -140,13 +140,13 @@ class BusDigital : public Bus {
 
   void setPixelColor(uint16_t pix, uint32_t c) {
     if (reversed) pix = _len - pix -1;
-    pix += _skip;
+    else pix += _skip;
     PolyBus::setPixelColor(_busPtr, _iType, pix, c, _colorOrder);
   }
 
   uint32_t getPixelColor(uint16_t pix) {
     if (reversed) pix = _len - pix -1;
-    pix += _skip;
+    else pix += _skip;
     return PolyBus::getPixelColor(_busPtr, _iType, pix, _colorOrder);
   }
 
diff --git a/wled00/data/settings_leds.htm b/wled00/data/settings_leds.htm
index 025e11a3c..635804dbe 100644
--- a/wled00/data/settings_leds.htm
+++ b/wled00/data/settings_leds.htm
@@ -350,7 +350,6 @@
 			<option value="2">Linear (never wrap)</option>
 			<option value="3">None (not recommended)</option>
 		</select><br>
-		Reverse LED order (rotate 180): <input type="checkbox" name="RV"><br>
     Skip first LED: <input type="checkbox" name="SL"><br>
     <span class="wc">
       Auto-calculate white channel from RGB:<br>
diff --git a/wled00/html_settings.h b/wled00/html_settings.h
index fec70fab7..412f117d9 100644
--- a/wled00/html_settings.h
+++ b/wled00/html_settings.h
@@ -134,9 +134,8 @@ required><br>Mode: <select name="TW"><option value="0">Wait and set</option>
 value="3">Sunrise</option></select><h3>Advanced</h3>Palette blending: <select 
 name="PB"><option value="0">Linear (wrap if moving)</option><option value="1">
 Linear (always wrap)</option><option value="2">Linear (never wrap)</option>
-<option value="3">None (not recommended)</option></select><br>
-Reverse LED order (rotate 180): <input type="checkbox" name="RV"><br>
-Skip first LED: <input type="checkbox" name="SL"><br><span class="wc">
+<option value="3">None (not recommended)</option></select><br>Skip first LED: 
+<input type="checkbox" name="SL"><br><span class="wc">
 Auto-calculate white channel from RGB:<br><select name="AW"><option value="0">
 None</option><option value="1">Brighter</option><option value="2">Accurate
 </option><option value="3">Dual</option><option value="4">Legacy</option>
diff --git a/wled00/set.cpp b/wled00/set.cpp
index a2bd03881..1f1b76bb4 100644
--- a/wled00/set.cpp
+++ b/wled00/set.cpp
@@ -126,14 +126,14 @@ void handleSettingsSet(AsyncWebServerRequest *request, byte subPage)
       }
       colorOrder = request->arg(co).toInt();
       start = (request->hasArg(ls)) ? request->arg(ls).toInt() : 0;
-      ledCount += length;
+//      ledCount += length;
 
       // actual finalization is done in loop()
       if (busConfigs[s] != nullptr) delete busConfigs[s];
       busConfigs[s] = new BusConfig(type, pins, start, length, colorOrder, request->hasArg(cv), skipFirstLed);
     }
 
-//    ledCount = request->arg(F("LC")).toInt();
+    ledCount = request->arg(F("LC")).toInt();
 //    if (t > 0 && t <= MAX_LEDS) ledCount = t;
     //DMA method uses too much ram, TODO: limit!