Loadpoint UI: add priority #20499
Loadpoint UI: add priority #20499
Conversation
added description link to emsesp
added PrioButton - effectivePriority used -> disableEdit - priority set -> show PrioButton
fix toml sort
|
Es gibt keine Very Low... very High Skala. Prio ist rein numerisch. |
|
Der Dropdown ist rein zur Übersicht very Low bis very High und nur in der UI. |
|
Die Prio ist aktuell numerisch. Dem jetzt andere Bezeichnungen zu geben finde ich verwirrend. Die Icons nutzen wir bereits für Phasenumschaltung und andere Mechanismen- die Doppelung wäre für mich ebenfalls verwirrend. Wie wäre es denn mit P1..10 stattdessen? Sinnvoll wäre auch anzuzeigen, ob ein Ladepunkt niedrigerer Priorität gerade übersteuert wird? |
|
Die beiden offenen Todos beziehen sich noch darauf, die Settings für die Ladepunkt Prio aus den Ladepunkt Settings zu entfernen und in den Ladepunkt Dialog zu bringen.
Was meinst du damit ? Die übersteuerung aus dem Fahrzeug ist berücksichtigt über die effectivePriority |
Da ist was dran. Gab schon mehrfach Diskussionen, wo Nutzer der Meinung waren Prio1 wäre die höchste Priorität. |
| :class="{ 'opacity-25': !editable }" | ||
| data-bs-toggle="tooltip" | ||
| data-bs-html="true" | ||
| :title="tooltipTitle" | ||
| :aria-label="editable ? $t('main.loadpointPrio.prioTooltip') : null" |
There was a problem hiding this comment.
| :class="{ 'opacity-25': !editable }" | |
| data-bs-toggle="tooltip" | |
| data-bs-html="true" | |
| :title="tooltipTitle" | |
| :aria-label="editable ? $t('main.loadpointPrio.prioTooltip') : null" |
There was a problem hiding this comment.
Der Tooltip sollte meiner Meinung nach auch im editierbaren modus erscheinen, da dieser schnell und kompakt die den echten numerischen Wert anzeigt (falls über die Icons nicht eindeutig)
There was a problem hiding this comment.
kompakt die den echten numerischen Wert anzeigt (falls über die Icons nicht eindeutig)
Wenn das wirklich ein Problem ist (bin ich noch nicht von überzeugt) müssen wir vielleicht doch noch mal über die Icon-Sprache nachdenken.
Ich bin auch, jetzt wo man es konkret im Code und in der Anwendung sieht, noch nicht so glücklich damit, dass es einmal nur eine Anzeige und einmal Veränderbar ist. Liegt an unserer ja doch etwas komplizierteren Logik (Fahrzeug kann Ladepunkt übersteuern), die ich bei der ersten Idee noch nicht so prominent auf dem Zettel hatte.
Vielleicht ist es doch einfacher, wenn die Prio nicht in-place editierbar ist, sondern wir direkt die Loadpoint-Settings (wie in deinem ersten Vorschlag) beim Klick öffnen.
Kommt der Wert vom Fahrzeug könnten wir beim Klick die Fahrzeug-Settings öffnen. Fahrzeug-Settings gibts (noch) nicht als eigenen Dialog. De-factor ist aber der "Ankunft"-Reiter im Ladeplan Dialog (wenn es ein Fahrzeug gibt) der Ort an dem wir schon heute Fahrzeug-Einstellungen editierbar (bspw. minsoc) haben. Hier könnten wir auch ein Prio-Setting hinzufügen (analog Ladepunkt). Dann könnte auch diese Einstellung im Config-UI (Fahrzeuge) entfernt werden. Wäre aus Symmetrie und Konsistenzgründe sicher gut.
Hat zudem den Vorteil, dass dem Nutzer damit deutlich klarer ist, ob er gerade was am Ladepunkt oder am Fahrzeug verändert. Die Anzeige im Main-UI wäre dann (wie ja im PR jetzt schon) immer nur die effectivePriority.
There was a problem hiding this comment.
Tooltip mit der Prio als Wert ist entfernt, es kommt nur noch der Hinweis dass es vom Fahrzeug übersteuert wird.
die Quick Priorisierung ist auch noch aktiv, sollen wir diese tatsächlich entfernen? per Loadpoint Config lässt diese sich nun im Dropdown style ändern.
There was a problem hiding this comment.
So, bin gerade mal die PR bei mir durchgegangen und hab einige kleinere Anpassungen direkt comittet. Ja, lass uns das Dropdown hier rausnehmen. Dann haben wir nicht mehr zwei Skalen (+3...-3) vs. die größere in den Settings. Lass uns Prio auch gleich in den Ankunft-Reiter bei bekannten Fahrzeugen mit aufnehmen.
Tooltip: Würde ich jetzt simple halten. Kein HTML, einfach "Priorität: +2". Vielleicht noch mit Ergänzung falls vom Fahrzeug übersteuert wurde: "Priorität: +2 (Fahrzeug)".
Den 0-Wert hab ich jetzt rausgenommen und auch die Logik, dass wir immer anzeigen oder gar nicht habe ich wieder reverted. Das ist sinnvoll wenn es ein direkt steuerbares Element ist, aber wenn das über die Settings gemacht wird können wir die 0-Zustand auch einfach weglassen.
created separated PriorityIcon from PrioritySelect added showPriorityOnAllLoadpoints (shows priority if a priority is set somewhere)
|
Hier nochmal ein paar Screenshots der Prio Zustände am Ladepunkt. 0 (default), -1 extreme Werte (Zahl + Richtung bei größer 3) Ladepunkt Einstellungen |
|
Müssen wir wirklich negative Prioritäten einführen? Das wäre schön zu vermeiden. Der Einmalaufwand z.B. 3 Ladepunkten eine "höhere" Prio zu geben damit der 4. auf 0 bleiben kann ist beherrschbar und kein Showstopper. Das wäre dann 0 (lowest) bis 10 (highest). Die hoch/runter Pfeile über der Zahl sind nicht sofort offensichtlich. Ich finde die doppelte Symbolverwendung der Phasenumschaltung bzw. ein/aus unglücklich. Da auch die Prio numerisch ist wäre für meinen Geschmack Einheitlichkeit klarer. Sollen die Prios permanent angezeigt werden? Genügt es i.S. Informationsoverload, nur übersteuerte Prios anzuzeigen? Würde es nicht sogar reichen zu zeigen, ob ein LP aufgrund seiner Prio gerade aktiv oder inaktiv wird? |
Doch, finde ich aus Nutzersicht wichtig. Ich hab hier vier Ladepunkte. Einer ist ein Heizstab. Der ist in meinem Kopf "low prio". Mir gehts hier nicht um dein einmaligen Config Aufwand die anderen höher zu stellen sondern darum, dass wir "Mental Model" besser abbilden. Default = "mittlere Prio" und abhängig vom Anwendungsfall kann hoch oder runterpriorisiert werden.
Wie die anderen Status-Icons auch wird hier nen Tooltip geben der erklärt. Ich find das mit der Zahl optisch schon sehr stark und unruhig. Daher würde ich für die gängigen Anwendungsfälle (wenige Ladepunkte) gerne eine dezentere Darstellung (ohne Zahl) haben. Hier ist mir Ruhe wichtiger als Konsistenz mit den "Extremen". Den Punkt mit der Phasenumschaltung sehe ich allerdings. Dort nutzen wir immer einen Doppelpfeil (hoch/runter). Hier mal ein Screenshot. Vielleicht ist ein guter Kompromiss bei den Prios nur bei einem einfachen Pfeil zu bleiben und die Zahl bei der
Ich fänd das gut, wenn wir das sinnvoll immer dann anzeigen wenn es für den Nutzer relevant ist. Mir fällt hier aber keine wirklich gute Logik ein an der wir das festmachen. Können wir aber auch in einer späteren Iteration immer noch "intelligenter" machen. |
Ich kann mich der Meinug von @andig nur anschliessen, ich finde -10 bis +10 einfach "too much". Logischer für mich wäre 0=default; 10 = highest.
Dann ist der Heizstab Prio=0, und die Ladepunkte einfach Prio 1, Prio 2 und z.B. Prio 10. Damit kann man immer noch den Heizstab auf Prio 5 ändern.... |
fixes #19650
Kompatibilität/Große Installationen