fix(/designs): studio-adjust summary precedence bug — state.hue||0!==0 parsed as state.hue || (0!==0) so 'changed' indicator was unreliable; now Object.keys(defaults).some() per-axis compare. Also 'on' → 'edited' for clarity

e850309495ddfdbae7c8946541d134224a8b05ec · 2026-05-14 09:33:26 -0700 · SteveStudio2

Files touched

M server.js

Diff

commit e850309495ddfdbae7c8946541d134224a8b05ec
Author: SteveStudio2 <stevestudio2@SteveStacStudio.lan>
Date:   Thu May 14 09:33:26 2026 -0700

    fix(/designs): studio-adjust summary precedence bug — state.hue||0!==0 parsed as state.hue || (0!==0) so 'changed' indicator was unreliable; now Object.keys(defaults).some() per-axis compare. Also 'on' → 'edited' for clarity
---
 server.js | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/server.js b/server.js
index e16c48c..fe13b0e 100644
--- a/server.js
+++ b/server.js
@@ -3942,8 +3942,12 @@ ${FOOTER}
       var f = buildFilter();
       var imgs = document.querySelectorAll('.design-card .card-img');
       for (var i=0;i<imgs.length;i++) imgs[i].style.filter = f;
-      var changed = state.hue||0!==0 || state.sat!==100 || state.bri!==100 || state.con!==100 || state.warm!==0 || state.sharp!==0;
-      if (summary) summary.textContent = changed ? 'on' : 'defaults';
+      // Fix precedence trap — prior version was `state.hue||0!==0` which parses
+      // as `state.hue || (0!==0)` so `changed` could be a number when hue!=0
+      // and was false when hue was 0 regardless of other sliders. Use explicit
+      // per-key comparison against defaults so summary always reflects truth.
+      var changed = Object.keys(defaults).some(function(k){ return state[k] !== defaults[k]; });
+      if (summary) summary.textContent = changed ? 'edited' : 'defaults';
     }
     function syncRow(row){
       var axis = row.getAttribute('data-axis');

← f69b164 feat(/designs): add 'Clear filters' chip when any hue/motif/ · back to Wallco Ai · chat: surface psql stderr + redact replicate token + throw o 4622641 →