Skip to content

Energyflow UI: expand solar/battery #20389

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

Merged
merged 1 commit into from
Apr 4, 2025
Merged

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Apr 4, 2025

relates to #9818

  • ☀️🔋 replaced multi pv/battery tooltip with expand collapse
  • 🏷️ use meter title for ui-configured devices (yaml entries get #.. fallback)
  • 💾 toggle positions (global, pv, battery) are stored in browser (local storage)
  • 👯 toggle only available if you have at lease devices

next steps (follow-up pr):
implement similar mechanism for load points and house (ext/aux)

large screen
Bildschirmfoto 2025-04-04 um 16 19 06

smaller screen
Bildschirmfoto 2025-04-04 um 16 19 13


video

expand.pv.battery.webm

@andig andig added the ux User experience/ interface label Apr 4, 2025
@andig andig merged commit dc57999 into master Apr 4, 2025
6 checks passed
@andig andig deleted the feature/expand_solar_battery branch April 4, 2025 16:16
@SolarPower2024
Copy link

SolarPower2024 commented Apr 5, 2025

@naltatis
Danke für die Implementierung 👍🏻

Im Querformat und bei größeren Bildschirmen ist das Top.
Ich finde aber bei kleineren Bildschirmen (Screenshot von einem Samsung Galaxy S24), sieht das aber etwas überladen aus (in Zusammenhang mit dem Forecast) bzw ist die Zeile mittlerweile so lange, dass das Wort "Erzeugung" schon nicht mehr vollständig Platz hat:
Screenshot_20250405_082635_evcc~2

Hättest du da vielleicht noch eine Idee, das in der Darstellung noch etwas zu optimieren?

@naltatis
Copy link
Member Author

naltatis commented Apr 5, 2025

Guter Punkt, gerade mit der Prognose wird das eng. Wir könnten mit Unterstreichung anstatt 🔻Icon arbeiten. Vielleicht auch nur auf dem kleinsten breakpoint wo Platz sehr begrenzt ist. Ich überlege mal ....

@TTXaser2
Copy link

Guter Punkt, gerade mit der Prognose wird das eng. Wir könnten mit Unterstreichung anstatt 🔻Icon arbeiten. Vielleicht auch nur auf dem kleinsten breakpoint wo Platz sehr begrenzt ist. Ich überlege mal ....

Kann mir jemand bitte erklären wie ich meine eigenen Namen da reinkrieg?

Mit "(yaml entries get #.. fallback)" kann ich schlicht und ergreifend nichts anfangen :-(
Was heisst das für mich als (noch) YAML-user? wo und wie genau muss ich da in die YAML was mit Hashtag # reinschreiben?

Danke für die Hilfe!

@naltatis
Copy link
Member Author

@TTXaser2 Meter Title sind nicht über die Yaml pflegbar. Du bekommst als "Fallback" dann Standardnamen (PV-Anlage #1..3) angezeigt. Eigene Namen funktionieren nur über Config UI erzeugte Meter. Hat technische Gründe wieso wir das nicht ohne riesigen Umbau auch in der yaml ermöglichen können.

@TTXaser2
Copy link

Vielen Dank für die schnelle Klärung!
Meine YAML ist ne große Baustelle mit viel ausprobieren, daher ist da recht oft was mit # Hashtag drin, vermutlich kommt daher meine Verwirrung :-/

@foersst

This comment was marked as off-topic.

@naltatis

This comment was marked as off-topic.

@cschlipf
Copy link
Contributor

Guter Punkt, gerade mit der Prognose wird das eng. Wir könnten mit Unterstreichung anstatt 🔻Icon arbeiten. Vielleicht auch nur auf dem kleinsten breakpoint wo Platz sehr begrenzt ist. Ich überlege mal ....

Vorschlag: Statt "Erzeugung" einfach "PV" als String nehmen? Oder gibt es hier auch Leute, die eine Windkraftanlage oder was anderes haben?

@premultiply
Copy link
Member

Ja, auch Dieselgeneratoren usw. gibt es 🫢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants