liquid-development/game-tracker
3
0
Fork 0
Code Issues 17 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 +67 -40
3 changed files with 67 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 10aad47124 - Show all commits

66
lib/presentation/views/main_menu/create_game/choose_game_view.dart
View File

@@ -35,42 +35,50 @@ class _ChooseGameViewState extends State<ChooseGameView> {
appBar: AppBar(
backgroundColor: CustomTheme.backgroundColor,
scrolledUnderElevation: 0,
leading: IconButton(
icon: const Icon(Icons.arrow_back_ios),
onPressed: () {
Navigator.of(
context,
).pop(selectedGameIndex == -1 ? null : selectedGameIndex);
},
),
title: const Text(
'Choose Game',
style: TextStyle(fontSize: 20, fontWeight: FontWeight.bold),
),
centerTitle: true,
),
body: Column(
children: [
CustomSearchBar(
controller: searchBarController,
hintText: 'Game Name',
),
const SizedBox(height: 5),
Expanded(
child: ListView.builder(
itemCount: widget.games.length,
itemBuilder: (BuildContext context, int index) {
return TitleDescriptionListTile(
title: widget.games[index].$1,
description: widget.games[index].$2,
badgeText: translateRulesetToString(widget.games[index].$3),
isHighlighted: selectedGameIndex == index,
onPressed: () async {
setState(() {
selectedGameIndex = index;
});
Future.delayed(const Duration(milliseconds: 500), () {
if (!context.mounted) return;
Navigator.of(context).pop(selectedGameIndex);
});
},
);
},
body: Container(
decoration: CustomTheme.standardBoxDecoration,
padding: const EdgeInsets.all(10),
child: Column(
children: [
CustomSearchBar(
controller: searchBarController,
hintText: 'Game Name',
flixcoo marked this conversation as resolved Outdated
sneeex commented 2025-12-07 17:08:14 +00:00
Outdated
Review
Copy Link

Es ist kein OnChanged implementiert, die search functionality ist insgesamt nicht gegeben hier.

Note: Das onChanged sollte aber vllt. auch required sein in der CustomSearchBar (weiß gerade nicht ob ich das aus nem grund optional gelassen habe, macht aber ja glaube keinen sinn oder?)

Es ist kein OnChanged implementiert, die search functionality ist insgesamt nicht gegeben hier. Note: Das onChanged sollte aber vllt. auch required sein in der CustomSearchBar (weiß gerade nicht ob ich das aus nem grund optional gelassen habe, macht aber ja glaube keinen sinn oder?)
flixcoo commented 2025-12-07 17:54:14 +00:00
Outdated
Review
Copy Link

Also ich habs explizit weggelassen, weil ich der einfachhalt halber das aktuell ausgesuchte Spiel über einen Index markiere. Wenn ich das jetzt implementieren würde, müsste ich dass unnötig kompliziert implementieren, weil die Games am Ende eh eine ID haben über die ich das markierte Identifizieren kann, deswegen würd ich das an dieser Stelle weglassen

Also ich habs explizit weggelassen, weil ich der einfachhalt halber das aktuell ausgesuchte Spiel über einen Index markiere. Wenn ich das jetzt implementieren würde, müsste ich dass unnötig kompliziert implementieren, weil die Games am Ende eh eine ID haben über die ich das markierte Identifizieren kann, deswegen würd ich das an dieser Stelle weglassen
sneeex commented 2025-12-07 17:59:39 +00:00
Outdated
Review
Copy Link

dann mach die searchbar weg, entweder searchbar und functionality oder nicht, aber nicht so halbe sachen.

dann mach die searchbar weg, entweder searchbar und functionality oder nicht, aber nicht so halbe sachen.
flixcoo commented 2025-12-07 18:31:54 +00:00
Outdated
Review
Copy Link

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen
sneeex commented 2025-12-07 18:53:26 +00:00
Outdated
Review
Copy Link

Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen

dann mach die functionality rein, entweder oder.
Du hast ja nur kein bock

> Ja aber das ist ja auch kacke, das layout an sich soll ja schon stehen dann mach die functionality rein, entweder oder. Du hast ja nur kein bock
),
),
],
const SizedBox(height: 5),
Expanded(
child: ListView.builder(
itemCount: widget.games.length,
itemBuilder: (BuildContext context, int index) {
return TitleDescriptionListTile(
title: widget.games[index].$1,
description: widget.games[index].$2,
badgeText: translateRulesetToString(widget.games[index].$3),
isHighlighted: selectedGameIndex == index,
onPressed: () async {
setState(() {
selectedGameIndex = index;
});
flixcoo marked this conversation as resolved Outdated
sneeex commented 2025-12-07 17:13:10 +00:00
Outdated
Review
Copy Link

warum ist unselecten hier nicht implementiert, beim choose group view aber schon?

warum ist unselecten hier nicht implementiert, beim choose group view aber schon?
flixcoo commented 2025-12-07 17:54:46 +00:00
Outdated
Review
Copy Link

ups, vergessen

ups, vergessen
},
);
},
),
),
],
),
),
);
}

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

@@ -33,6 +33,16 @@ class _ChooseGroupViewState extends State<ChooseGroupView> {
appBar: AppBar(
backgroundColor: CustomTheme.backgroundColor,
scrolledUnderElevation: 0,
leading: IconButton(
icon: const Icon(Icons.arrow_back_ios),
onPressed: () {
Navigator.of(context).pop(
selectedGroupIndex == -1
? null
: widget.groups[selectedGroupIndex],
);
},
),
title: const Text(
'Choose Group',
style: TextStyle(fontSize: 20, fontWeight: FontWeight.bold),
@@ -46,12 +56,11 @@ class _ChooseGroupViewState extends State<ChooseGroupView> {
return GestureDetector(
flixcoo marked this conversation as resolved
sneeex commented 2025-12-07 17:40:33 +00:00
Review
Copy Link

Warum lässt sich in Choose Game View suchen, im Choose Group View aber nicht? Können beides ja potenziell sehr lange Listen sein

Warum lässt sich in Choose Game View suchen, im Choose Group View aber nicht? Können beides ja potenziell sehr lange Listen sein
flixcoo commented 2025-12-07 18:13:02 +00:00
Review
Copy Link

Erledigt (So würde dass dann auch für die Games funktionieren)

Erledigt (So würde dass dann auch für die Games funktionieren)
onTap: () {
setState(() {
selectedGroupIndex = index;
});
Future.delayed(const Duration(milliseconds: 500), () {
if (!context.mounted) return;
Navigator.of(context).pop(widget.groups[index]);
if (selectedGroupIndex == index) {
selectedGroupIndex = -1;
} else {
selectedGroupIndex = index;
}
});
},
child: GroupTile(

20
lib/presentation/views/main_menu/create_game/choose_ruleset_view.dart
View File

@@ -36,6 +36,16 @@ class _ChooseRulesetViewState extends State<ChooseRulesetView> {
appBar: AppBar(
backgroundColor: CustomTheme.backgroundColor,
scrolledUnderElevation: 0,
leading: IconButton(
sneeex marked this conversation as resolved Outdated
sneeex commented 2025-11-25 22:05:04 +00:00
Outdated
Review
Copy Link

Macht keinen Sinn, da gametype und rulesets ausgewählt werden können

Macht keinen Sinn, da gametype und rulesets ausgewählt werden können
flixcoo commented 2025-11-26 11:28:55 +00:00
Outdated
Review
Copy Link

Habs jetzt zu Choose Ruleset geändert, da man mit den Gametypes ja auch im Prinzip nur n Ruleset festlegt

Habs jetzt zu `Choose Ruleset` geändert, da man mit den Gametypes ja auch im Prinzip nur n Ruleset festlegt
icon: const Icon(Icons.arrow_back_ios),
onPressed: () {
Navigator.of(context).pop(
selectedRulesetIndex == -1
? null
: widget.rulesets[selectedRulesetIndex].$1,
);
},
),
title: const Text(
'Choose Ruleset',
style: TextStyle(fontSize: 20, fontWeight: FontWeight.bold),
sneeex marked this conversation as resolved Outdated
sneeex commented 2025-11-25 22:05:36 +00:00
Outdated
Review
Copy Link

warum so viele comments? ist doch klar wofür die settings jeweils sind, also label indicator, divider etc

warum so viele comments? ist doch klar wofür die settings jeweils sind, also label indicator, divider etc
flixcoo commented 2025-11-26 11:29:34 +00:00
Outdated
Review
Copy Link

Wollte das einfach bisschen klarer aufteilen

Wollte das einfach bisschen klarer aufteilen
@@ -49,11 +59,11 @@ class _ChooseRulesetViewState extends State<ChooseRulesetView> {
return TitleDescriptionListTile(
onPressed: () async {
setState(() {
selectedRulesetIndex = index;
});
Future.delayed(const Duration(milliseconds: 500), () {
if (!context.mounted) return;
Navigator.of(context).pop(widget.rulesets[index].$1);
if (selectedRulesetIndex == index) {
selectedRulesetIndex = -1;
} else {
selectedRulesetIndex = index;
}
});
},
title: widget.rulesets[index].$2,