MatchSummaryTile durch MatchTile ersetzen #142
sneeex
Reference in New Issue
Block a user
Delete Branch "enhancement/137-matchsummarytile-durch-matchtile-ersetzen"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
MatchSummaryTile durch MatchTile ersetzen
Zugehörige Issue(s):
Closes #137
Beschreibung
Die
MatchSummaryTileimHomeViewwurden durch eine kompakte Variante derMatchTiles ersetzt. Diese wurden dafür angepasst.Änderungen
MatchTileentfernt, damit dieser beim benutzen gesetzt werden kann.compact-Flag zumMatchTilehinzugefügt, die eine kompaktere Variante anzeigtMatchTiles hinzugefügt, welche keinenwinnergesetzt habenMatchTiles inHomeViewintegeriertVergleich
Gefällt mir garnicht actually, erkennt man garnicht, dass es sich um die letzten zwei Matches handelt, macht für mich keinen Sinn so, das muss schon in ner Box sein
also das wirklich toxic
Ja ich checke, aber ich finde viel wichtiger, dass diese match tiles sich dort mit den match tiles in match view zusammenpassen. Willst du sonst mal ausprobieren?
pack die tiles mal in ein container wie vorher, also replace quasi wirklich nur die alten, dann vielleicht passt
warum 3 tiles und nicht 2?
und warum hier so gelöst und nicht vom platz als wären zwei in dem container? das sieht doch so voll leer aus.
find ich garnicht so schlecht vom design her
Jaja er hat's auch geändert jetzt, aber dieses issue meinte ich nicht xD
Habe drei gemacht, weil dadurch dass die kleiner geworden sind wollte ich mehr vom Bildschirm ausfüllen.
Und das empty Tile wollte ich nicht so unendlich groß machen, wenn es eh nur aussagt, dass nichts da ist
aber findest du nicht, dass 3 kacke aussieht bzw. keins eben auch, wenn das so wenig platz braucht?
Ich checke was du meinst, aber ich fände dann die bessere Alternative das Tile garnicht anzuzeigen. Aber das wär auch irgendwie blöd, weil man dann als User am anfang garnicht checkt warum das so leer ist
ich finde 2 glaube ich ganz gut. Also so wie es jetzt ist
Aber jetzt hast du die Match history gefickt xD
Ups hä, schau ich mir an
Dings @sneeex ich hab dein kommentar vorher falsch verstanden, es sind sogar 2 Tiles, nur die skeleton data hatte noch 3 tiles, ist gefixt
ja so schauts gut aus :)
der abstand ist nicht mehr der gleiche
Ich hab den Abstand an den gleichen wie die Tile-Breite in HomeView angepasst, also 95% der verfügbaren Breite. Du hattest das vorher über Margin gelöst, deswegen ist das jetzt unterschiedlich
findet ihr das match sollte clickable sein im homescreen?
hab gerade ausversehen draufgeklickt
aber euch störts nicht, dass es so leer ist dann?
Ja safe, also aktuell vllt noch nicht so sinnvoll, weil es ja nur einen screen dahinter gibt, aber finde das sollte so, um da dann schnell hinzukommen. Das ist ja der gedanke dahinter, nicht erst zu suchen
Also optimal find ichs auch nicht, aber lieber so als dass das Tile unnötig groß mit keinen sinnvollen informationen drin ist. Ggf. erweitern wir da ja auch noch inhalte drauf
Wenn man auf eins der Tiles klickt und winner auswählt und dann zurück geht, aktualisiert der HomeScreen nicht mti dem Winner/ keinem Winner
Ja, wenn wir nen komplettes Match View haben mit Name, Spieler, Gruppe, Score usw, dann macht das ja sinn
kein skeleton hier? also bzgl dem state fix
das update des winners wenn man zurück geht, ist im matchview gefühlt auch schneller als im home view
und bitte adaptivePageRoute
außerdem lädst du alle home view daten neu, obwohl nur winner gechanged haben kann?
Habs geupdated, skeleton in diesem fall nicht nötig, weil der change ja nicht aus der db kommt
skeleton nicht nötig weil der change ja nicht aus der db kommt ???!?!?!?
Was ist das dann hier?
Nvm nicht drüber nachgedacht, aber fände trotzdem unnötig, nur wegen so einer kleinen änderung
machen wir aber überall doch so?
Ja weil wir aktuell auch noch kein Caching haben, ich find das generell auch nicht so schön aber speziell an dieser Stelle ists ja auch einfach unnötig