liquid-development/game-tracker
3
0
Fork 0
Code Issues 16 Pull Requests 6 Actions Releases 1 Activity

CreateGameView erstellen #67

Merged
sneeex merged 35 commits from feature/3-creategameview-erstellen into development 2025-12-10 20:43:54 +00:00
Conversation 28 Commits 35 Files Changed 10 +2 -2
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 062c2681bf - Show all commits

4
lib/presentation/views/main_menu/create_game/choose_group_view.dart
View File

@@ -76,8 +76,8 @@ class _ChooseGroupViewState extends State<ChooseGroupView> {
visible: filteredGroups.isNotEmpty,
replacement: const TopCenteredMessage(
icon: Icons.info,
title: 'No group',
message: 'There is no group matching your search.',
title: 'Info',
message: 'There is no group matching your search',
),
flixcoo marked this conversation as resolved
sneeex commented 2025-12-08 13:39:52 +00:00
Review
Copy Link

Hier bitte ohne Punkt, das hatten wir ja gesagt, und wir müssen uns darauf einigen ob der Titel infos erhält, oder nur keywords wie info/error, weil ich das bis jetzt so gemacht habe.
Sonst passt so

Hier bitte ohne Punkt, das hatten wir ja gesagt, und wir müssen uns darauf einigen ob der Titel infos erhält, oder nur keywords wie info/error, weil ich das bis jetzt so gemacht habe. Sonst passt so
flixcoo commented 2025-12-08 16:24:53 +00:00
Review
Copy Link

Wenn der Titel keine Infos enthalten soll, würd ich daraus n enum Attribut machen damit der je nach enum gesetzt wird (Icon und Titel)

Wenn der Titel keine Infos enthalten soll, würd ich daraus n enum Attribut machen damit der je nach enum gesetzt wird (Icon und Titel)
sneeex commented 2025-12-08 16:46:44 +00:00
Review
Copy Link

ja findste mit oder ohne richtigen Title besser?

ja findste mit oder ohne richtigen Title besser?
sneeex commented 2025-12-08 16:49:00 +00:00
Review
Copy Link

ist mir auch egal, lass es so oder nicht. Wenn du es mit Info/Error im Titel willst, mach am besten das einmal temporär in den Titel und dann nen Issue für Änderung der anderen Titel in eine enum Version oder halt eine Version mit aussagekräftigem Titel.

Aber auf jeden Fall den Punkt weg machen am Ende von Message
dann approve ich

ist mir auch egal, lass es so oder nicht. Wenn du es mit Info/Error im Titel willst, mach am besten das einmal temporär in den Titel und dann nen Issue für Änderung der anderen Titel in eine enum Version oder halt eine Version mit aussagekräftigem Titel. **Aber auf jeden Fall den Punkt weg machen am Ende von Message** dann approve ich
flixcoo commented 2025-12-09 17:26:15 +00:00
Review
Copy Link

erledigt

erledigt
child: ListView.builder(
padding: const EdgeInsets.only(bottom: 85),