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.

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

UI: adaptive power digits #13619

Merged
merged 2 commits into from
Apr 28, 2024
Merged

UI: adaptive power digits #13619

merged 2 commits into from
Apr 28, 2024

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Apr 28, 2024

fixes #13585

  • ๐Ÿ“ show to kW digits if values are below 10 kW
  • ๐Ÿ‘ฏโ€โ™€๏ธ w/kw and digit are always konsistant inside each visual group: visualization chart (top), details table (below)
  • ๐Ÿ”‹ change also affects energy values like chargedEnergy

xxx W individual values below 1 kW
below 1kw

x.xx kW individual values in visualization chart below 10 kW
below 10kw

x.xx kW individual values in visualization chart and table below 10 kW
below 10kw everywhere

xx.x kW individual values above 10 kW
note: visualization bar still shows higher precision (x.xx kW) since non of the bar-parts is above 10kW
above 10kw

@naltatis naltatis added enhancement New feature or request ux User experience/ interface labels Apr 28, 2024
@andig
Copy link
Member

andig commented Apr 28, 2024

Der zweite Screenshot zeigt nicht was die รœberschrift aussagt?

@naltatis
Copy link
Member Author

Ich hab noch mal einen weiteren Screenshot eingefรผgt. Beim zweiten war im oberen Bereich x.xx kW und im unteren xx.x kW.

@andig
Copy link
Member

andig commented Apr 28, 2024

Rein damit?

@naltatis naltatis merged commit a6056e2 into master Apr 28, 2024
7 checks passed
@naltatis naltatis deleted the feature/three_digits branch April 28, 2024 21:11
naltatis added a commit that referenced this pull request Apr 30, 2024
naltatis added a commit that referenced this pull request Apr 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Leistungsanzeige: 3 signifikante Stellen
2 participants