Reduzierung der #defines, stattdessen Einstellungen in Web-UI

a) Für was sind die Farben gelb und blau?

Exemplarisch hier 2 zusätzliche Kontroll-LEDs, siehe Statische WS2812 LEDs . Da sollte in der Web-UI sicher noch ein Text „Kontroll-LED“ davor und ein schönerer Farbpickerl.

b) Die Apostrophe müssen aus dem UI noch raus :slight_smile:

Mit der UI-Umsetzung bin ich ja noch nicht ganz glücklich & bitte hier um Mithilfe. Weil es so viel ist bezieht sich mein PR zunächst nur auf das Backend.
Aber das lässt sich bestimmt umsetzen;-)

c) Kannst du, wenn du die LED-Anpassung jetzt machst, auch gleich NEOPIXEL_REVERSE_ROTATION mit reinnehmen?

Oh ja, zusätzlich fehlt auch noch LED_OFFSET. Reiche ich noch nach…

Ich glaube das ist tatsächlich gar nicht verkehrt, da es viel Übersicht bietet, wenn man später mal was sucht oder erweitern will.

Die globalen Variablen sind bereits recht verstreut, evt. schiebt man noch ein Refactoring hinterher, sollte dann nicht so schwer sein. Vom Laden/Speichern rücke ich aber erstmal nicht ab, das funktioniert perfekt ohne neue Bugs. Init() muss eh aufgerufen werden um z.B. die geänderte Anzahl von LEDs neu zu setzen.

2 „Gefällt mir“