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 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
c1b76a801d
commit
4ff73e5110
@@ -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] },
|
||||
|
||||
Reference in New Issue
Block a user