Implementierung der Games #203
3 Participants
Notifications
Due Date
No due date set.
Blocks
#213 Neuer Regelsatz: Platzierung
liquid-development/game-tracker
Reference: liquid-development/game-tracker#203
Reference in New Issue
Block a user
Delete Branch "feature/119-implementierung-der-games-2"
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?
Zugehörige Issue(s)
Closes #119
Beschreibung
Neu aufgesetzte Variante von #177
Es wurde das erstellen von Spielvorlagen und das Hinzufügen dieser zu Spielen hinzugefügt. Außerdem wurden hinzugefügt, Games an diversen Stellen anzuzeigen und Konsistenzänderungen implementiert.
Änderungen
Game.iconist nicht mehr required, Standardmäßig auf leeren stringGame.colorist standardmäßigGameColor.orangesowohl in der Model-Klasse als auch in derCreateGameViewgesetzt.CreateGameViewist standardmäßig aufsingleWinnergesetztChooseTiles in derCreateGameViewwurden Popup-Dropdowns hinzugefügt, durch welche sich Farbe und Regelwerk auswählen lassen.ChooseGameViewwurde überarbeitet, so dass die Farben der Spielvorlagen auch dargestellt werden.Neue Widgets
GameLabelhinzugefügt, welches auf denMatchTilesund imMatchDetailViewdie Spielvorlage + Regelsatz anzeigt.GameTilefür die Auswahl der Spielvorlage implementiertTitleDescriptionListTilewurde simplifiziert.Neue Ansichten
CreateGameViewum eine neue Spielvorlage zu erstellen oder eine bestehende zu bearbeitenKonsistenz
Zusätzliche Anmerkungen
No response
flixcoo referenced this pull request2026-04-28 13:29:48 +00:00
WIP: Implementierung der Gamesto Implementierung der Games@@ -357,2 +389,4 @@"delete": "Delete","delete_all_data": "Delete all data","delete_game": "Delete Game","delete_game_with_matches_warning": "If you delete this game template, {count, plural, =1{1 match} other{{count} matches}} using this game template will also be deleted.",hinter
otherein space vergessen?Kenne mich nicht mit den templates aus
Das ist richtig so, wenn count =1 dann wird
1 matchzurückgegeben, und in allen anderen fällen{count} matches.@@ -155,0 +164,4 @@/// **'Color'**String get color;/// No description provided for @color_blue.sind die descriptions hier nicht mies unnötig?
ja, ich entfern die mal
@@ -272,6 +362,12 @@ abstract class AppLocalizations {/// **'Error while creating group, please try again'**String get error_creating_group;/// Error message when game deletion failsbitte weniger vibe coden felix 🤓 🫵
hä was meinst du? :D
@gelbeinhalb wie gefallen dir sonst die UI changes, dazu irgendwelche meinung?
Klickbarer bereich für color selection bei game edit/creation ist nur der color text, nicht die farbe an sich und auch nicht das icon rechts neben color, finde das ux technisch sehr weird. Also ich meine der klickbare Bereich um das dropdown zu öffnen, nicht um die farben auszuwählen
Edit: würde glaube ich tatsächlich den ganzen balken klickbar machen oder?
Anderer Fehler:
Tritt auf wenn man Game in Match Creation auswählt und dann aber während es ausgewählt ist löscht
finde glaube text zu klein oder? also im vergleich zu allem anderen (d.h. den buttons und dem titel)
Habe aber auch irgendwie das gefühl es ist noch zu "leicht" das game template zu löschen wenn dadurch viele (bzw > 0) games gelöscht werden. Wie wärs mit sowas wie gib "delete" ein oder so. Also ich finde ein game template zu löschen mit so vielen assoziierten games ist schon krass und ja auch nicht recoverable oder? oder machen wir da auch nur deleted attribut?
Ja könnte man noch drüber nachdenken, ob man das vllt mit nem delay im button enabled noch verbessert, würds jetzt aber erstmal so lassen
Es ist leider nicht so einfach, weil ich die position des popups nicht einstellen kann und wenn ich das über die gesamte choose tile machen würde, das popup halt auch mittig wäre und das sieh m.M.n net so nice aus. Hab mal was gepusht gucks dir gerne an
aber der button rechts oder das icon ist immer noch nicht klickbar, das auch weird? also sonst okay m.m.n zumindest erstmal
Habs jetzt nochmal überarbeitet, jetzt ist zumindest das chevron auch klickbar
wenn keine games da, soll message wie no games created yet angezeigt werden, natürlich auch lokalisiert
aber der screen sieht gut aus finde ich
leck doch meine eier fett, warum so viel code
@@ -64,3 +64,3 @@static BoxDecoration highlightedBoxDecoration = BoxDecoration(color: boxColor,border: Border.all(color: primaryColor),border: Border.all(color: textColor, width: 2),warum ist die border hier die textcolor? checks nicht so vom prinzip, weil in dem game view sind die highlighted border ja in der jeweiligen game farbe und nicht der text farbe
Das hat nichts mit der
ChooseGameViewzutun, ich wollte einfach nur das Highlighting von Containern ändern.@@ -344,0 +351,4 @@.getSingle();return count ?? 0;}würde man das nicht eher in game dao packen? weil es geht doch um games und die damit assozierten matches und nicht andersrum, das steht auch im string da falsch
Nein, ich will die Anzahl an Matches mit einer spezifischen Game-ID. Ich frage ja auch den Match Table an, deswegen ists in der matchDao
@@ -481,0 +496,4 @@final query = delete(matchTable)..where((m) => m.gameId.equals(gameId));final rowsAffected = await query.go();return rowsAffected;}hier genauso?
Ich arbeite auf der Match-Table, deswegen ists in der matchDao
@@ -25,0 +41,4 @@"type": "int"}}},im singular: werden 1 spiel gelöscht? dat passt ja net oder
fixed
@@ -0,0 +214,4 @@// Choose ruleset tileif (!isEditMode())ChooseTile(meinste nicht, dass man die tiles hier nicht lieber alle auslagern sollte? die sind arg lang
Ja das ding ist die werden halt nur einmal verwendet
ja oder wenigstens ausgelagert in der datei?
ich kanns sonst als funktion unten drunter packen?
Ja genau
@@ -32,3 +33,4 @@final VoidCallback? onWinnerChanged;final VoidCallback? onMatchesUpdated;ist das das callback wenn ein match im edit view/create view gemacht wurde und dann in der liste aktualisiert werden soll?
das ist der callback für die MatchDetailView an erster stelle, um dort das Match zu aktualisieren
ne das ist doch on match update, on matches updated ist doch was anderes?
Ja onMatchUpdate ist für ein einzelnes Match und onMatchesUpdate für alle Matches im MatchView
@@ -26,3 +29,4 @@onPressed: onPressed,buttonText: text,buttonType: buttonType,isDescructive: isDestructive,was macht das?
Das macht den Button auf Rot
@@ -262,0 +302,4 @@gameId: testGame.id,);expect(count, 2);});oben testest du auch noch löschen hier nicht? wieso?
fixed