Closed deingithub closed 5 months ago
Ich mag deine Design-Änderungen und hatte selbst auch schon länger einen Vermerk im Todo-Berg, die an weiteren Stellen zu übernehmen. Insofern bin ich sehr dankbar für diesen PR :)
Wahrscheinlich werde ich die einzelnen Änderungen nach und nach in travelynx/main cherry-picken. Außer ich finde spontan die Zeit den kompletten PR auf einmal zu bearbeiten. Mal schauen!
Soo, jetzt ist alles drin.
Bei der History habe ich die Reihenfolge vertauscht, so dass erst Ziel und dann Start gelistet werden. Die ursprüngliche Variante wirkt von den Icons her zwar etwas stimmiger, beißt sich aber mit der Sortierung der History: Wenn einzelne Fahrten von neu nach alt sortiert sind sollten IMHO auch die Ziele von neu (Ziel) nach alt (Start) sortiert sein und nicht andersrum.
Soweit zumindest meine aktuelle Einschätzung, mal gucken ob ich die Änderung doch noch rückgängig mache oder uns etwas anderes einfällt.
In jedem Fall: Vielen lieben Dank :)
Alles sehr gerne ^^
Das mit der History-Sortierung ist so tatsächlich ein Problem und die Sortierung einheitlich zu haben ist wahrscheinlich weniger verwirrend. Ich würde bei dieser Reihenfolge aber eher andere Icons vorschlagen, das place
wirkt oben etwas out of place:
Das Origin-Icon wäre play_circle_filled
in Kombination mit einem stumpfen
a.origin .material-icons {
transform: rotate(-90deg);
}
und dann als Destination-Icon radio_button_unchecked
.
Eventuell wäre stattdessen ein Pfeil statt der „Perlenschnur“-Linie sinnvoll, ich weiß aber spontan nicht wie ich den in CSS umsetzen würde.
Hmmmja, guter Punkt. Ich probiere da die Tage mal ein bisschen herum :)
Also ich würde schon erwarten, dass oben Start & unten Ziel steht, so ist man es ja auch von Karten-Apps Routing oder vom Bahn.expert routing gewöhnt. Ich verstehe deinen Punkt allerdings auch @derf und habe gerade auch keinen besseren Vorschlag. Wenn mir was einfällt, lass ich es euch wissen! Danke aber ganz generell für dieses super Update!
Kleiner Bug: es sollte für die Icons eine Safe Area geben, ansonsten bricht der Text (wenn zu lang) in diese Ions um:
Guter Punkt @Ein-Tim, vielleicht ist das ganze mit einer <ul>
und list-style
besser umgesetzt als jetzt gerade, kann mich auch da dran machen oder es dir überlassen @derf :smile:
@deingithub mach gerne, das geht wahrscheinlich schneller als auf mich zu warten
Moin, anbei ein Konvolut aus Bugfixes und Vorschlägen, der Reihe nach:
_format_train
Vorlage bekommt die Farbkodierung aus der Abfahrtstafel (#93) und wird jetzt an den meisten Stellen wie auch in der Statuscard und der History-Liste verwendet,Gerade was den letzten Commit angeht hoffe ich, es ist in Ordnung, so mit der Tür ins Haus zu fallen. Es hat sich so im Flow ergeben. Wenn das allgemein ein Problem ist, lass ich's in Zukunft und mach grundsätzlich erstmal ein Issue auf. Ich hätte da noch ein paar weitere Ideen für die UI, wollte aber auch keinen zu großen PR aus dem Nichts senden. Wenn diese Änderung spezifisch nicht passt, sollte sie relativ sauber trennbar von den Fixes sein. Änderungswünsche nehme ich natürlich auch gerne an :sweat_smile: