Skip to content

Koala fixes#2558

Merged
benderl merged 6 commits intoopenWB:masterfrom
benderl:koala-fixes
Jul 21, 2025
Merged

Koala fixes#2558
benderl merged 6 commits intoopenWB:masterfrom
benderl:koala-fixes

Conversation

@benderl
Copy link
Contributor

@benderl benderl commented Jul 15, 2025

fix codestyle of some PRs

@benderl benderl self-assigned this Jul 15, 2025
@benderl benderl requested a review from Brett-S-OWB July 15, 2025 09:12
@benderl benderl added this to the 2.1.8 milestone Jul 15, 2025
Copy link
Contributor

@Brett-S-OWB Brett-S-OWB left a comment

Choose a reason for hiding this comment

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

disable history chart: mit { duration: 0 } Ist die Animation immer noch vorhanden, nur auf einen Zeitraum von 0 Sekunden gesetzt. Um das Animation richtig auszumachen, muss die Animation auf Falsch gesetzt werden. Dann braucht man den entsprechenden Type von chart.js, um Typefehler zu vermeiden.

time charging enable toggle position: unnötige CSS und divs (zu viele). Toggle switch rechtsbündig. Obwohl ich war unsicher ob rechtsbündig hier besser wäre, da es hat ein bisschen komisch ausgesehen wenn der Zeitplan button direkt darunter ist/war aber das ist wahrscheinlich Geschmack Sache.

duplicate code: DC current conversion: wiederholte Kalkulation DC zu Leistung und Leistung to DC ausgelagert in separate Funktion um Wiederholung zu vermeiden.

style definition in BaseTable: CSS zu Style Abschnitt umgezogen für Lesbarkeit

vehicle soc module check: Auch wenn in den Einstellungen kein SoC-Modul zugewiesen ist, existieren die Topics unter dem SoC-Modul trotzdem – inklusive des config-Topics mit type: null, aber ohne name. Daher dachte ich es wäre sicherer, auf name === undefined zu prüfen. type: null hatte ich zwar überlegt, aber falls type z. B. als leerer String zurückkommt, wäre der boolesche Wert nicht Korrekt. Deshalb habe ich mich für name === undefined entschieden aber wenn das backend immer null zurück gibt ist es sauberer.

standard theme legend visibility: Mit diesem neuen Container kann die gesamte Legende gezielt per JavaScript ein- oder ausgeblendet werden ($('#legend-container').show();), z.B. erst wenn das Chart fertig geladen ist.

@benderl benderl merged commit 5c34342 into openWB:master Jul 21, 2025
1 check passed
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.

2 participants