Repository navigation
Conversation
There was a problem hiding this comment.
🟢 Approval recommended
The only noted issue is a non-blocking nit concerning label consistency in the legacy theme.
No open issues 🎉
What changed in this PR
This pull request renames the battery priority mode from “Fahrzeuge” to “Verbraucher” and updates its visual indicators across the Koala and Cards themes.
Changes:
- Renames the
ev_modelabel and tooltip to “Verbraucher”. - Adds vehicle/consumer icon combinations to battery controls and status displays.
- Updates the Cards battery-mode modal with the matching label and plug icon.
| File | Summary |
|---|---|
packages/modules/web_themes/koala/source/src/composables/useBatteryModes.ts |
Renames the shared battery mode label and tooltip. Review note: the still-supported standard_legacy theme retains the “Fahrzeuge” label for the same setting. |
packages/modules/web_themes/koala/source/src/components/BatteryModeButtons.vue |
Displays vehicle and power-consumer icons for the renamed mode. |
packages/modules/web_themes/koala/source/src/components/BatteryCard.vue |
Adds the combined consumer-mode icon to battery cards. |
packages/modules/display_themes/cards/source/src/components/Dashboard/FlowCard.vue |
Adds the combined vehicle/consumer indicator to the dashboard flow card. |
packages/modules/display_themes/cards/source/src/components/Battery/BatteryModeModal.vue |
Updates the mode label and adds the plug icon to the modal. |
Review effort: Lite
|
@cshagen Möchtest Du das im Colors-Theme auch anpassen? Die Priorität gilt nun für Fahrzeuge und Verbraucher. |
|
Ich würde das gerne anpassen, aber mir ist nicht klar, wie die neue Logik funktioniert. Gibt es neu drei Optionen für die Priorität - Speicher, Fahrzeuge oder Verbraucher? Oder sind bei den Fahrzeugen jetzt immer die Verbraucher mit dabei?
…
Am 26.10.2026 um 15:38, LKuemmel ***@***.***> schrieb:
LKuemmel left a comment (openWB/core#4192)
@cshagen Möchtest Du das im Colors-Theme auch anpassen? Die Priorität gilt nun für Fahrzeuge und Verbraucher.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS and Android. Download it today!
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
| outline | ||
| :color="mode.color != 'dark' ? mode.color : 'light'" | ||
| :active="mode.value === mqttStore.getBatteryMode" | ||
| :title="mode.label" |
There was a problem hiding this comment.
Ein Tooltip, der den selben Inhalt wie das Label des Buttons hat, macht eigentlich keinen Sinn.
| :title="mode.label" |
| <template v-if="mode.value === 'ev_mode'"> | ||
| / | ||
| <font-awesome-icon | ||
| fixed-width | ||
| :icon="['fas', 'plug']" | ||
| /> | ||
| </template> |
There was a problem hiding this comment.
Diese Spezialbehandlung im Template passt nicht zu dem System mit den Daten in mode.
Besser wäre es, dass allgemein aufzubohren und mode.symbol in ein Array zu überführen.
Ich erstelle einen Commit dafür.
| v-else | ||
| :x="svgSize.circleRadius * 1.45" | ||
| :y="-svgSize.circleRadius * 1.35" | ||
| :height="svgSize.circleRadius * 0.6" |
There was a problem hiding this comment.
Siehe Kommentar zu BatteryModeModal.vue.
| <template v-if="batteryMode.value === 'ev_mode'"> | ||
| <span class="q-mx-xs">/</span> | ||
| <q-icon name="power" size="sm" color="primary" /> | ||
| </template> |
There was a problem hiding this comment.
Siehe Kommentar zum Cards Theme.
| <q-icon :name="mode.icon" /> | ||
| <template v-if="mode.value === 'ev_mode'"> | ||
| <span class="q-mx-xs">/</span> | ||
| <q-icon name="power" /> | ||
| </template> | ||
| <span class="q-ml-xs">{{ mode.label }}</span> |
There was a problem hiding this comment.
Siehe Kommentar zum Cards Theme.
An den Modi hat sich nichts geändert. Lediglich die Bezeichnung von |
|
replaced by #4197 auch für das Colors-Theme |
Uh oh!
There was an error while loading. Please reload this page.