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.
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.... |
naltatis
added
backlog
|
Sehe ich genauso, negative Prioritäten sind nicht notwendig. Habe selbst viele Geräte und auch einen Heizstab der mental auf niedrigster prio ist. |
|
Bei mir in der Arbeit gabs auch immer Missverständnisse bzgl Prio und numerischen Werten, ich habe dann mal die Begriffe 'Important' und 'Not Important' dafür eingeführt und seitdem war alles klar. Vielen Dank für Eure super Arbeit hier! Ich frei mich schon, wenn meine Wärmepumpe und der Heizstab hoffentlich diesen Herbst noch Realität werden :-) |
7 participants
fixes #19650
Kompatibilität/Große Installationen