-
-
Notifications
You must be signed in to change notification settings - Fork 518
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
Target SoC Slider / Loadpoint Layout Cleanup #835
Conversation
Moin Michael, könnten wir das in kleinere Schritte zerlegen? Hier passieren verschiedene Dinge gleichzeitig die ich noch nicht voll verstehe? |
Jein, ich hab schon probiert den Scope hier nicht zu weit zu öffnen. Ist mir leider nur teilweise gelungen. Das hängt leider alles etwas zusammen. Für Target-Charge ist es halt wichtig, dass man immer die Möglichkeit hat einen WunschSoC zu wählen. Dafür ist die bisherige SoC Auswahl eher ungeeignet. Ich wollte aber auf keinen Fall einen zweiten SoC-Wähler einführen. Daran hingt dann der Layoutumbau der Loadpoints um hier a) den Platz zu schaffen und b) neue Redundanzen zu vermeiden. Was ich anbieten kann ist, dass ich das die eigentliche UI für das Wählen/Aktivieren/Deaktivieren der Zielzeit hier rausziehe und dann später in einen neuen PR auslagere. Dann würde dieser PR quasi die UI-Vorbedingung für Target-Charge sein. SOC-Wahl und Layoutumbau würde hier verbleiben. Modal/API-Call/Zeitanzeige wandert raus. Macht das Sinn? Schafft dann auch etwas Raum um ggf. an der Ladesteuerung noch zu optimieren. Alternativ kannst du natürlich auch hier Fragen an den Groß-PR schreiben die ich kommentiere und erkläre. |
Habe das gerade mal für ein erstes Testing gemerged. Der neue TargetSoC Slider ist super klasse, auch dass er in 5% Schritten springt.
Musste für mich einige merge Konflikte auflösen, will einen Seiteneffekt auf das obig beschriebene Verhalten nicht ganz ausschließen. |
Ich hab den Target-Time Dialog hier jetzt erstmal rausgenommen um den PR etwas kleiner zu halten und hier erstmal "nur" den Target SoC Wähler und die Loadpoints Vereinfachung live zu bringen. Die Progressbar inkl. Beschriftung ist auch noch etwas größer geworden. @mark-sch Eine 8/4 Aufteilung passt an der Stelle nicht weils dann die Fluchten mit den Site-Details bricht. Aber du hast in deinem Fork ja scheinbar noch nen vierten Eintrag mit den Verbrauchsdaten. Im Original sind die Elemente bei der Auflösung dann auch bündig. Zeitselektion verwendet native Browser UI. Ob da 5-min Schritte unterstützt werden hängt vom Browser und OS ab. Die mobilen Geräte machen das aktuell besser als die Desktop-Varianten. Btw was hast du eigentlich mit unserem schicken evcc Blitz ⚡️ oben links gemacht? 🙃 |
Ich wars nicht! Das war @mark-sch 🙃 |
Whoops. Falscher Person geantwortet. |
Ahh, es gibt noch Unterschiede durch das native Browser UI, verstehe. Mit dem Zielladen sah aber sonst schon gut aus, ein paar Dinge sind mir dann bei backend seitiger Integration noch aufgefallen. Nimmst du es mit einem separaten branch wieder auf oder bleibt die Funktion erstmal on hold? |
Ja, das Zielladen kommt dann in einem neuen PR sobald dieser hier durch ist. Gerade weil ja noch ein paar Fragen offen sind (bspw Persistenz) ist das ne gute Idee das erstmal zu entkoppeln. |
Ich persönlich finde diese Meldung ja ziemlich groß und eigentlich auch unnötig. |
Ja sehe ich auch so. Hab's jetzt erstmal wieder so reingenommen wie es vorher war damit das Feature nicht verloren geht. Ein Icon (Evtl mit tooltip) wäre sicher eleganter und platzsparender. Dass dann aber in einem neuen PR. |
Neues Update. Habe noch mal an den Aufteilungen gedreht.
großer screen mittlerer screen kleiner screen |
Geilomat! Find ich richtig Klasse!!! |
Ich auch. Wird immer noch besser. |
Wirklich toll, freue mich vor allem schon auf den Ladetimer im nächsten PR :-) . Gibt es hier noch offene Punkte? |
What changed?
target charge time can be set using a modal (max 7 days in the future, using native browser time input)moved vehicle range closer to the progress bar and vehicle nameTodos:
[ ] verify target charge works reliably (for me it often didn't reach the desired soc state at the set time)smaller screen
.larger screen
vehicle soc