liquid-development/game-tracker
3
0
Fork 0
Code Issues 14 Pull Requests 7 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 +30 -20
1 changed files with 30 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 7cff48ebc0 - Show all commits

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

@@ -3,6 +3,7 @@ import 'package:game_tracker/core/custom_theme.dart';
import 'package:game_tracker/data/dto/group.dart';
import 'package:game_tracker/presentation/widgets/text_input/custom_search_bar.dart';
import 'package:game_tracker/presentation/widgets/tiles/group_tile.dart';
import 'package:game_tracker/presentation/widgets/top_centered_message.dart';
class ChooseGroupView extends StatefulWidget {
final List<Group> groups;
@@ -71,26 +72,35 @@ class _ChooseGroupViewState extends State<ChooseGroupView> {
),
),
Expanded(
child: ListView.builder(
padding: const EdgeInsets.only(bottom: 85),
itemCount: filteredGroups.length,
itemBuilder: (BuildContext context, int index) {
return GestureDetector(
onTap: () {
setState(() {
if (selectedGroupId != filteredGroups[index].id) {
selectedGroupId = filteredGroups[index].id;
} else {
selectedGroupId = '';
}
});
},
child: GroupTile(
group: filteredGroups[index],
isHighlighted: selectedGroupId == filteredGroups[index].id,
),
);
},
child: Visibility(
visible: filteredGroups.isNotEmpty,
replacement: const TopCenteredMessage(
icon: Icons.info,
title: 'No group',
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),
itemCount: filteredGroups.length,
itemBuilder: (BuildContext context, int index) {
return GestureDetector(
onTap: () {
setState(() {
if (selectedGroupId != filteredGroups[index].id) {
selectedGroupId = filteredGroups[index].id;
} else {
flixcoo marked this conversation as resolved Outdated
sneeex commented 2025-12-07 19:04:04 +00:00
Outdated
Review
Copy Link

hier noch etwas einblenden wenn kein suchergebnis da ist, also filteredGroups leer ist (bspw. topcenteredmessage)

hier noch etwas einblenden wenn kein suchergebnis da ist, also filteredGroups leer ist (bspw. topcenteredmessage)
selectedGroupId = '';
}
});
},
child: GroupTile(
group: filteredGroups[index],
isHighlighted:
selectedGroupId == filteredGroups[index].id,
),
);
},
),
),
),
],