From 4ff73e5110a5264b8757e3b367715e068d2d5328 Mon Sep 17 00:00:00 2001 From: tajniak81 <13187254+tajniak81@users.noreply.github.com> Date: Thu, 3 Sep 2026 21:01:01 +0200 Subject: [PATCH] Brightness and the solar floor become sliders too Both are a place on a short, known range, which is what the slider row added for the current limit is for. Typing 70 into a box that only accepts tens was the worse way to say it. The solar minimum gets no floor note under it. The current limit's says that below six amps the charger pauses rather than charging slowly, which is a sentence about a ceiling; this is the least a solar charge will draw, and the same words would be wrong about it. Main breaker limit stays a box. Ten to five hundred amps is too wide a range to aim at with a slider. Co-Authored-By: Claude Opus 5 --- Web App/web/src/views/Charging.vue | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/Web App/web/src/views/Charging.vue b/Web App/web/src/views/Charging.vue index 821044a..46d7e5f 100644 --- a/Web App/web/src/views/Charging.vue +++ b/Web App/web/src/views/Charging.vue @@ -654,7 +654,10 @@ const MQTT_SETTING_BLOCKS = [ fields: [ { key: "solarBalancing", at: "solarBalancing", label: "solarBalancing", type: "switch" }, { key: "solarChargeMode", at: "settings.solarChargeMode", label: "solarChargeMode", type: "option", enum: "solarMode", values: [0, 1] }, - { key: "solarMinCurrentA", at: "settings.solarMinCurrentA", label: "solarMinCurrent", type: "number", min: LIMIT_FLOOR, max: 32, step: 1, unit: "A" }, + // A slider, like the current limit it shares a floor with. No floor note + // under it though: this is the least a solar charge will draw, not a + // ceiling, so the limit slider's hint would be saying the wrong thing. + { key: "solarMinCurrentA", at: "settings.solarMinCurrentA", label: "solarMinCurrent", type: "slider", min: LIMIT_FLOOR, max: 32, step: 1, unit: "A" }, // This command offers automatic and single-phase only. The three-phase // setting is a Modbus register, and offering it here would be offering a // write that comes back refused. @@ -666,7 +669,9 @@ const MQTT_SETTING_BLOCKS = [ id: "panel", title: "blockPanel", fields: [ - { key: "ledBrightness", at: "ledBrightness", label: "ledBrightness", type: "number", min: 0, max: 100, step: 10, unit: "%" }, + // A slider, like the current limit: a brightness is a place on a range, + // and typing 70 into a box that only takes tens is a worse way to say it. + { key: "ledBrightness", at: "ledBrightness", label: "ledBrightness", type: "slider", min: 0, max: 100, step: 10, unit: "%" }, { key: "lightOffSchedule", at: "settings.lightOffSchedule", label: "lightOff", type: "switch" }, { type: "window", label: "lightOffWindow", from: "lightOffStart", to: "lightOffEnd", at: ["settings.lightOffStart", "settings.lightOffEnd"] }, { key: "swipeUpMode", at: "swipeUpMode", label: "swipeUp", type: "option", enum: "gesture", values: [0, 1, 2, 3] },