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

Mobile UI: Active loadpoint indicator #12973

Merged
merged 3 commits into from
Mar 15, 2024
Merged

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Mar 15, 2024

fixes #12961

  • shows lighting instead of dot for load points that are charging.
  • distinguish between connected/not connected

⚡= charging
● = connected
○ = not connected

all connected, 2nd charging & selected
Bildschirmfoto 2024-03-15 um 14 10 11

all connected, 2nd charging, 1st selected
Bildschirmfoto 2024-03-15 um 14 10 07

not connected, connected & selected, not connected, connected
Bildschirmfoto 2024-03-15 um 15 10 54

not connected, charging & selected, not connected, connected
Bildschirmfoto 2024-03-15 um 15 11 55

@naltatis naltatis added enhancement New feature or request ux User experience/ interface labels Mar 15, 2024
@naltatis naltatis requested a review from andig March 15, 2024 13:17
@andig
Copy link
Member

andig commented Mar 15, 2024

Uiiiiiiiih- nice! Wenn wir schonmal dabei sind- vllt. auch noch einen kleinen Stecker für "connected"?

@naltatis
Copy link
Member Author

Würde erstmal sagen "keep it simple". Der Connected Status ist ja etwas complicated (Steckdosen, Wärmepumpen, ...).

@andig
Copy link
Member

andig commented Mar 15, 2024

Naja, Status ist Status. Wenn ich bei 4 LP durcheinander komme (was ja hier das Problem ist) dann fände ich es auch nett, den Verbundenen zu sehen? #12961 spricht von "aktivem" Ladepunkt, zeigt als Gegenbeispiel aber nur den nicht verbundenen. Aber entscheide gerne Du das...

@naltatis
Copy link
Member Author

Naja, Status ist Status.

Ich will das gar nicht ausschließen. Aber die "Integrated Devices" als "connected" anzuzeigen, greift vmtl. zu kurz. Die sind aufgrund ihrer Natur immer verbunden und da würde das Steckersymbol nur Unruhe (im Vergleich zum Punkt) reinbringen.
Heißt wir sollten den "connected" Status nur bei "normalen" Ladepunkten nutzen, wo dieser Zustand auch wirklich ne sinnvolle Information liefert.

Ich schau mal wie viel "Sonderlocke" das ist ...

@naltatis
Copy link
Member Author

@andig Für Stecker-Icons sind die Punkte zu filigran, das sieht sonst total schnell bastelig/wuselig aus. Ich hab jetzt mal mit der Füllung des Punktes gearbeitet: ● = connected, ○ = not connected. Das ist nicht so selbsterklärend wie der Blitz und für neue Nutzer evtl. etwas verwirrend. Sieht aber clean aus und ist etwas, was man durch Beobachtung rausfinden kann.

An das Status-Konzept der Integrated Devices / Schaltsteckdosen müssen wir gesondert noch mal ran. Das sich der abhängig vom Modus ändert ist auch nicht intuitiv (auch wenn ich verstehe warum wir das aktuell machen). Aber anderes Thema.

@andig
Copy link
Member

andig commented Mar 15, 2024

Klasse, danke dass Du nochmal rein geschaut hast! Wäre vllt. auch ein Erwähnung im Blog wert. Langsam sollten wir dann mal fürs nächste große Release sammeln.

@andig andig merged commit 8ad4022 into master Mar 15, 2024
6 checks passed
@andig andig deleted the feature/active_lp_indicator branch March 15, 2024 15:09
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 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.

Mobile UI: Aktiven Ladepunkt an erste Stelle setzen
2 participants