From 18c886cbc58b6e26a99be2f7d01192be766b5856 Mon Sep 17 00:00:00 2001 From: "alexei.dolgolyov" Date: Fri, 13 Mar 2026 16:05:37 +0300 Subject: [PATCH] Use primary color for running node icons and fix add device modal dirty check - Running graph nodes show entity icon in --primary-color instead of --text-muted - Fix AddDeviceModal always showing dirty: serialize zones array with JSON.stringify for proper strict equality comparison (matching DeviceSettingsModal pattern) Co-Authored-By: Claude Opus 4.6 --- server/src/wled_controller/static/css/graph-editor.css | 5 +++++ .../wled_controller/static/js/features/device-discovery.js | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/server/src/wled_controller/static/css/graph-editor.css b/server/src/wled_controller/static/css/graph-editor.css index 2193b7f..e914e42 100644 --- a/server/src/wled_controller/static/css/graph-editor.css +++ b/server/src/wled_controller/static/css/graph-editor.css @@ -305,6 +305,11 @@ opacity: 0.5; } +.graph-node.running .graph-node-icon { + stroke: var(--primary-color); + opacity: 0.85; +} + /* ── Running indicator (animated gradient border) ── */ .graph-node.running .graph-node-body { diff --git a/server/src/wled_controller/static/js/features/device-discovery.js b/server/src/wled_controller/static/js/features/device-discovery.js index 83acece..4f610c0 100644 --- a/server/src/wled_controller/static/js/features/device-discovery.js +++ b/server/src/wled_controller/static/js/features/device-discovery.js @@ -28,7 +28,7 @@ class AddDeviceModal extends Modal { baudRate: document.getElementById('device-baud-rate').value, ledType: document.getElementById('device-led-type')?.value || 'rgb', sendLatency: document.getElementById('device-send-latency')?.value || '0', - zones: _getCheckedZones('device-zone-list'), + zones: JSON.stringify(_getCheckedZones('device-zone-list')), zoneMode: _getZoneMode(), }; }