Conversation
Brett-S-OWB
left a comment
There was a problem hiding this comment.
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.
fix codestyle of some PRs