Skip to content

Active bat control UI reordering / Config description text#958

Open
Brett-S-OWB wants to merge 17 commits into
openWB:mainfrom
Brett-S-OWB:improve-active-bat-control-UI
Open

Active bat control UI reordering / Config description text#958
Brett-S-OWB wants to merge 17 commits into
openWB:mainfrom
Brett-S-OWB:improve-active-bat-control-UI

Conversation

@Brett-S-OWB
Copy link
Copy Markdown
Contributor

Überarbeitung der aktiven Speichersteuerung zur Verbesserung der Übersichtlichkeit und Bedienbarkeit:
• Das Regellimit wurde aus der Hauptübersicht entfernt und kontextabhängig als Unterpunkt der jeweiligen Regelbedingung integriert (Anzeige nur bei Bedarf)
• Die Preisgrenzen-Regelung wurde strukturell angepasst, um die Bedienung klarer und nachvollziehbarer zu gestalten
• Ergänzung einer dynamischen Verhaltensbeschreibung, die basierend auf Regelbedingung, Speichersteuerung und Regelmodus eine Zusammenfassung der aktuellen Konfiguration anzeigt

Hinweis:
Branch basiert auf einem Stand vor dem Merge der Umbenennungen:
Active bat Control: Improve Text #954.
Merge-Konflikte ergeben sich daraus, dass diese Änderungen inzwischen in main enthalten sind.
Rebase und Konfliktauflösung erfolgen nach dem aktuellen Release im Zuge des Patch-Branches.

Konzept:
Bildschirmfoto 2026-04-16 um 17 27 30

Screen shots:
Bildschirmfoto 2026-04-16 um 17 39 02

Bildschirmfoto 2026-04-16 um 17 39 23 Bildschirmfoto 2026-04-16 um 17 39 39

@LKuemmel LKuemmel requested a review from ndrsnhs April 17, 2026 06:34
@LKuemmel LKuemmel added this to the 2.2.0-Patch.1 milestone Apr 28, 2026
@LKuemmel LKuemmel self-requested a review April 28, 2026 09:39
Copy link
Copy Markdown
Contributor

@LKuemmel LKuemmel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bei Preisgrenze bitte noch die Beschriftungen, wo die Preise eingestellt werden, aktualisieren.
Bei Fahrzeugladung aktiv kann nur Regelmodus anwenden ausgewählt werden, die beiden anderen Optionen sollen wie in dem Entwurf ausgegraut werden, da sie keine sinnvolle Konfiguration sind, aber für eine konsistente Darstellung angezeigt werden sollen.

Comment thread src/views/ActiveBatControlConfiguration.vue Outdated
@Brett-S-OWB
Copy link
Copy Markdown
Contributor Author

Bei Preisgrenze bitte noch die Beschriftungen, wo die Preise eingestellt werden, aktualisieren. Bei Fahrzeugladung aktiv kann nur Regelmodus anwenden ausgewählt werden, die beiden anderen Optionen sollen wie in dem Entwurf ausgegraut werden, da sie keine sinnvolle Konfiguration sind, aber für eine konsistente Darstellung angezeigt werden sollen.

Eingabe Title angepasst:
Eigenregelung für Strompreise über x ct/kWh &
Speicher aktiv laden für Strompreise unter y ct/kWh

Bei Regelbedingung - Fahrzeugladung aktiv - Button "Eigenregelung" & "Speicher aktiv laden" disabled

Bildschirmfoto 2026-04-28 um 15 43 30

@benderl benderl requested review from LKuemmel and removed request for ndrsnhs April 29, 2026 09:22
@LKuemmel
Copy link
Copy Markdown
Contributor

LKuemmel commented May 5, 2026

Die ja/nein-Buttons bei der Preisgrenze waren im Entwurf nicht mehr drin. Bitte mit @ndrsnhs klären, ob wir die wirklich noch brauchen.

@Brett-S-OWB
Copy link
Copy Markdown
Contributor Author

Aus den aktuell zwei Schaltern wurde einen einzigen Aktivierungsschalter gemacht um es Nutzer freundlicher zu machen.
Commit : [dd6fb39]

Bildschirmfoto 2026-05-05 um 17 31 47
Bildschirmfoto 2026-05-05 um 17 31 56

@benderl
Copy link
Copy Markdown
Contributor

benderl commented May 6, 2026

Rein vom UI her hast Du das gut gelöst. Da die beiden Topics jetzt aber immer synchron sein müssen, macht eigentlich nur ein einzelnes Topic Sinn.

Ich habe in dem PR openWB/core#3349 das Topic "bat_control_permitted" aus dem Backend entfernt. Auch dort wurden die eigentlich immer zusammen ("and") betrachtet.

Bitte auf den Feature-Branch anpassen und auch ein Rebase, damit der Konflikt behoben wird. Dann kann der PR eigentlich rein.

@benderl benderl self-assigned this May 6, 2026
@benderl benderl added enhancement New feature or request core depends on changes in core repository labels May 6, 2026
@Brett-S-OWB Brett-S-OWB force-pushed the improve-active-bat-control-UI branch from 65028e3 to b369209 Compare May 6, 2026 12:16
@Brett-S-OWB
Copy link
Copy Markdown
Contributor Author

Rein vom UI her hast Du das gut gelöst. Da die beiden Topics jetzt aber immer synchron sein müssen, macht eigentlich nur ein einzelnes Topic Sinn.

Ich habe in dem PR openWB/core#3349 das Topic "bat_control_permitted" aus dem Backend entfernt. Auch dort wurden die eigentlich immer zusammen ("and") betrachtet.

Bitte auf den Feature-Branch anpassen und auch ein Rebase, damit der Konflikt behoben wird. Dann kann der PR eigentlich rein.

Angepasst and rebased

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

Labels

core depends on changes in core repository enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants