minor fixess

webconfig/api.py

@@ -217,7 +217,7 @@ def api_set_layout():
     data = request.get_json(force=True, silent=True) or {}
     cfg = load_config()
     layout = cfg.setdefault("layout", {})
-    for k in ("mode", "columns", "spacing", "showLabels"):
+    for k in ("mode", "columns", "spacing", "showLabels", "labelColor"):
         if k in data:
             layout[k] = data[k]
     save_config(cfg)
@@ -243,10 +243,10 @@ def api_set_feedback():
     data = request.get_json(force=True, silent=True) or {}
     cfg = load_config()
     fb = cfg.setdefault("feedback", {})
-    for k in ("okColor", "errorColor"):
+    for k in ("okColor", "errorColor", "holdColor", "pendingColor"):
         if k in data:
             fb[k] = data[k]
-    for k in ("okWidth", "errorWidth", "glowRadius"):
+    for k in ("okWidth", "errorWidth", "glowRadius", "pendingWidth"):
         if k in data:
             try:
                 fb[k] = int(data[k])
@@ -260,11 +260,23 @@ def api_set_feedback():
 @login_required
 def api_set_settings():
     data = request.get_json(force=True, silent=True) or {}
+    nested = data.get("settings")
+    if isinstance(nested, dict):
+        data = {**data, **nested}
     cfg = load_config()
     st = cfg.setdefault("settings", {})
+    if not isinstance(st, dict):
+        st = {}
     for k in ("darkMode", "password", "kioskMode", "iconsDir", "webPort"):
-        if k in data:
-            st[k] = data[k]
+        if k not in data:
+            continue
+        val = data[k]
+        if k == "password":
+            if not isinstance(val, str) or not val.strip():
+                abort(400, "password required")
+            val = val.strip()
+        st[k] = val
+    cfg["settings"] = st
     save_config(cfg)
     return jsonify({"ok": True})