search

evcc-io / evcc

Solar Charging ☀️🚘
https://evcc.io
MIT License
3.63k stars 674 forks source link

Config UI: device value formatting #17258

Closed naltatis closed 2 days ago

naltatis commented 1 week ago

addition to https://github.com/evcc-io/evcc/pull/17179

Bildschirmfoto 2024-11-14 um 12 03 23

Bildschirmfoto 2024-11-14 um 12 03 29

Bildschirmfoto 2024-11-14 um 12 03 38

andig commented 1 week ago

Wir könnten auch kürzer schreiben Current L1...3 ?

naltatis commented 1 week ago

Wir könnten auch kürzer schreiben Current L1...3 ?

@andig Ist in diesem PR ja bewusst auf die , Schreibweise geändert worden um es klarer zu machen.

Wir kommunizieren in Wiederholungen 😄

Es gab den Vorschlag, den ich auch gut und richtig finde, das auszuschreiben. Daher habe ich das geändert https://github.com/evcc-io/evcc/pull/17179/commits/dc1dd2a326732fee64dbfe0edfbb8fc8e21f9b0c. Siehe https://github.com/evcc-io/evcc/discussions/17153#discussioncomment-11211058 Und ja, das kann damit jetzt zu Auspunktung auf kleinen Größen führen. Finde ich aber nicht so schlimm und ist eher ein Edgecase.

andig commented 1 week ago

Wir kommunizieren in Wiederholungen

Ja, das liegt daran dass die Antwort nicht eindeutig war ;) Mich stört die Inkonsistenz links/rechts.

naltatis commented 1 week ago

Ich hab auch überlegt links diesen Mittelpunkt zu verwenden. Das ist dann zwar konsistent sieht aber sehr komisch/ungewohnt aus.

andig commented 1 week ago

Ja, daher der "hübsche" Vorschlag: L1...3. Oder: Phase Currents, Powers etc.