Skip to content

Koala - Battery Priority SoC range slider#3218

Merged
benderl merged 11 commits intoopenWB:masterfrom
Brett-S-OWB:battery-charge-range
Mar 20, 2026
Merged

Koala - Battery Priority SoC range slider#3218
benderl merged 11 commits intoopenWB:masterfrom
Brett-S-OWB:battery-charge-range

Conversation

@Brett-S-OWB
Copy link
Contributor

Range-Slider für den SoC-Bereich der Speicher-Ladepriorität hinzugefügt.

Bildschirmfoto 2026-03-18 um 17 02 36

@Brett-S-OWB Brett-S-OWB force-pushed the battery-charge-range branch from 7dfa11d to f3c24a0 Compare March 19, 2026 07:03
@Brett-S-OWB
Copy link
Contributor Author

Beim Range-Slider verwende ich ein anderes Rot für den Pending-Text, da das bisherige Rot auf dem blauen Label-Hintergrund schlecht lesbar ist. Die Frage wäre ob wir dieser Rot überall verwenden wollen?

Bildschirmfoto 2026-03-19 um 08 07 49 Bildschirmfoto 2026-03-19 um 08 07 20 Bildschirmfoto 2026-03-19 um 08 09 06

@Brett-S-OWB Brett-S-OWB requested a review from benderl March 19, 2026 07:21
@benderl
Copy link
Contributor

benderl commented Mar 19, 2026

Beim Range-Slider verwende ich ein anderes Rot für den Pending-Text, da das bisherige Rot auf dem blauen Label-Hintergrund schlecht lesbar ist. Die Frage wäre ob wir dieser Rot überall verwenden wollen?

Wie wäre es alternativ, das komplette Label einzufärben oder nur den "thumb"? Die beiden Rottöne finde ich auf dem blauen Hintergrund nicht gut lesbar.

@Brett-S-OWB
Copy link
Contributor Author

Beim Range-Slider verwende ich ein anderes Rot für den Pending-Text, da das bisherige Rot auf dem blauen Label-Hintergrund schlecht lesbar ist. Die Frage wäre ob wir dieser Rot überall verwenden wollen?

Wie wäre es alternativ, das komplette Label einzufärben oder nur den "thumb"? Die beiden Rottöne finde ich auf dem blauen Hintergrund nicht gut lesbar.

Ja das wäre ein Alternative, ich schaue wie das aussieht...

@Brett-S-OWB
Copy link
Contributor Author

Bei Slider wird Label-Hintergrund rot bei "pending" Wert statt den Text.

Bildschirmfoto 2026-03-19 um 12 06 42

@benderl
Copy link
Contributor

benderl commented Mar 19, 2026

Spricht etwas gegen die bei Quasar eingebauten Optionen für die Farbe der Labels?
https://quasar.dev/vue-components/range#other-customizations

@Brett-S-OWB
Copy link
Contributor Author

Spricht etwas gegen die bei Quasar eingebauten Optionen für die Farbe der Labels? https://quasar.dev/vue-components/range#other-customizations

Nein das wäre natürlich besser :)
Die Pending-Markierung wurde auch angepasst, sodass die Label-Farbe abhängig davon geändert wird, ob der linke oder rechte Slider bewegt wurde.

Bildschirmfoto 2026-03-19 um 14 02 52

@benderl benderl merged commit 543bce8 into openWB:master Mar 20, 2026
2 checks passed
@seaspotter
Copy link
Contributor

ksnip_20260323-151211

Bei mir sieht das aktuell so aus, ich habe trotzdem noch ein Scrollmenü, weil die 100% SoC vom rechten Slider über den Fensterrand hinausschauen.

Zudem habe ich Min SoC 97% und Max SoC 100% eingestellt, ist so leider nicht voneinander unterscheidbar, aber ich kann beide verändern. Macht es ggf. Sinn Min und Max auf zwei Slider aufzuteilen wie es in den Einstellungen auch ist?
Was vielleicht noch etwas unschön ist und ggf. gar nicht so gewollt: Ich kann den Max SOC unterhalb des Min SOC schieben, da sollte vielleicht noch eine Prüfung rein?

@Brett-S-OWB
Copy link
Contributor Author

Bei mir sieht das aktuell so aus, ich habe trotzdem noch ein Scrollmenü, weil die 100% SoC vom rechten Slider über den Fensterrand hinausschauen.

Danke für den Hinweis. Ich musste das Vue UI Range-Component mit CSS anpassen damit die Labels unterhalb des Sliders sind. da fehlt noch ein bisschen Padding Rechts bei slider auf 100%. Ich passe das noch an.

Zudem habe ich Min SoC 97% und Max SoC 100% eingestellt, ist so leider nicht voneinander unterscheidbar, aber ich kann beide verändern. Macht es ggf. Sinn Min und Max auf zwei Slider aufzuteilen wie es in den Einstellungen auch ist?

Ja, stimmt – ein Label verdeckt das andere, wenn sie so nah beieinander sind. Das zu ändern, ohne das Basis-UI-Component stark zu verändern, wird schwierig. Wenn man die Thumbs anklickt, sieht man den aktuellen Wert, sodass man ihn trotzdem problemlos einstellen kann. Die Änderungen sieht man dann auch in der Speicherkarte, nachdem die Werte gesetzt wurden:

Bildschirmfoto 2026-03-23 um 17 50 28

Was vielleicht noch etwas unschön ist und ggf. gar nicht so gewollt: Ich kann den Max SOC unterhalb des Min SOC schieben, da sollte vielleicht noch eine Prüfung rein?

Falls man die Slider-Thumbs übereinander bewegt (also wenn Max und Min quasi auf der „falschen“ Seite liegen), wird der Thumb auf der linken Seite des Sliders immer als Minimum behandelt und der Thumb auf der rechten Seite immer als Maximum – egal, wie sie zuvor hin- und hergeschoben wurden. Dieses Verhalten finde ich persönlich intuitiv und in Ordnung.

@seaspotter
Copy link
Contributor

Ja, stimmt – ein Label verdeckt das andere, wenn sie so nah beieinander sind. Das zu ändern, ohne das Basis-UI-Component stark zu verändern, wird schwierig. Wenn man die Thumbs anklickt, sieht man den aktuellen Wert, sodass man ihn trotzdem problemlos einstellen kann. Die Änderungen sieht man dann auch in der Speicherkarte, nachdem die Werte gesetzt wurden:

Danke dir für die Rückmeldung. Ich persönlich fände 2 Slider je einen für Min und einen für Max SOC besser, weil es dann konsistent mit den Einstellungen im Menü wäre. :) Aber reine persönliche Meinung, ich kann auch damit leben wie es jetzt ist. :)

ksnip_20260323-212146

Falls man die Slider-Thumbs übereinander bewegt (also wenn Max und Min quasi auf der „falschen“ Seite liegen), wird der Thumb auf der linken Seite des Sliders immer als Minimum behandelt und der Thumb auf der rechten Seite immer als Maximum – egal, wie sie zuvor hin- und hergeschoben wurden. Dieses Verhalten finde ich persönlich intuitiv und in Ordnung.

Okay super, das hatte ich noch gar nicht getestet. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants