Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Config UI: device value formatting #17258

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Config UI: device value formatting #17258

merged 2 commits into from
Nov 20, 2024

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Nov 14, 2024 •
edited
Loading

addition to #17179

  • 🔘 dot-seperation and single unit for power values
  • ↔ improve space allocation between label and value for specific fields

Bildschirmfoto 2024-11-14 um 12 03 23

Bildschirmfoto 2024-11-14 um 12 03 29

Bildschirmfoto 2024-11-14 um 12 03 38

@naltatis naltatis added enhancement New feature or request ux User experience/ interface experimental Experimental feature labels Nov 14, 2024
@andig
Copy link
Member

andig commented Nov 14, 2024

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

@naltatis
Copy link
Member Author

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 dc1dd2a. Siehe #17153 (comment)
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
Copy link
Member

andig commented Nov 14, 2024 •
edited
Loading

Wir kommunizieren in Wiederholungen

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

@naltatis
Copy link
Member Author

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

@andig
Copy link
Member

andig commented Nov 14, 2024

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

@naltatis naltatis merged commit 5bfa661 into master Nov 20, 2024
5 checks passed
@naltatis naltatis deleted the fix/device_values_power branch November 20, 2024 09:43
mabunixda added a commit to mabunixda/evcc that referenced this pull request Nov 22, 2024
@mabunixda
Merge branch 'master' into local/patched
cbdecb6 
* master: (58 commits)
  Add Huawei EMMA (evcc-io#17338)
  Config UI: device value formatting (evcc-io#17258)
  chore: upgrade npm dependencies (evcc-io#17344)
  chore: fix template
  Easee: fix PhaseGetter returning used, not configured, phases  (evcc-io#17326)
  chore: refactor
  MacOS: add gobuildid
  Script: simplify setters
  Tariffs: formula, charges, tax > advanced fields (evcc-io#17301)
  Push: add custom messenger (BC) (evcc-io#17211)
  Script: add missing string setter (evcc-io#17314)
  Sofar: fix docs (evcc-io#17324)
  Polestar: skip test
  Polestar: fix authentication (evcc-io#17276)
  chore: fix line breaks
  chore: fix quotes
  Add Tessie (evcc-io#17274)
  PUN: update api endpoint (evcc-io#17270)
  Revert "EM24: add pv usage & fix energy (evcc-io#17173)"
  Enphase: fix soc
  ...
@SolarPowerEV
Copy link
Contributor

SolarPowerEV commented Nov 26, 2024 •
edited
Loading

Eine Idee kam mir gerade:
Was wäre, wenn man die Darstellung etwas umstellt und so strukturiert:
Current              L1 12.3 · L2 13.4 · L3 33.5 A

oder alternativ noch mit Doppelpunkt:
Current              L1: 12.3 · L2: 13.4 · L3: 33.5 A

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request experimental Experimental feature ux User experience/ interface
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@naltatis @andig @SolarPowerEV