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

CreateGroupView erstellt #28

Merged
flixcoo merged 37 commits from feature/5-creategroupview-erstellen into development 2025-11-19 17:32:44 +00:00
Conversation 13 Commits 37 Files Changed 16 +309
1 changed files with 309 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit a54495f915 - Show all commits

309
lib/presentation/views/main_menu/create_group/create_group_view.dart Normal file
View File

@@ -0,0 +1,309 @@
import 'package:flutter/material.dart';
import 'package:game_tracker/core/custom_theme.dart';
import 'package:game_tracker/data/db/database.dart';
import 'package:game_tracker/data/dto/player.dart';
import 'package:game_tracker/presentation/widgets/full_width_button.dart';
import 'package:game_tracker/presentation/widgets/top_centered_message.dart';
import 'package:provider/provider.dart';
import 'package:skeletonizer/skeletonizer.dart';
class CreateGroupView extends StatefulWidget {
const CreateGroupView({super.key});
@override
State<CreateGroupView> createState() => _CreateGroupViewState();
}
class _CreateGroupViewState extends State<CreateGroupView> {
List<Player> selectedPlayers = [
Player(id: '0', name: 'Player 0'),
Player(id: '0', name: 'Player 0'),
Player(id: '0', name: 'Player 0'),
Player(id: '0', name: 'Player 0'),
];
late Future<List<Player>> _allPlayersFuture;
late final List<Player> skeletonData = List.filled(
7,
Player(id: '0', name: 'Player 0'),
);
@override
@override
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:26:01 +00:00
Outdated
Review
Copy Link

id nach #35 nicht mehr notwendig

id nach #35 nicht mehr notwendig
void initState() {
super.initState();
final db = Provider.of<AppDatabase>(context, listen: false);
_allPlayersFuture = db.playerDao.getAllPlayers();
}
@override
Widget build(BuildContext context) {
addSamplePlayers(context);
return SafeArea(
child: Scaffold(
backgroundColor: CustomTheme.backgroundColor,
appBar: AppBar(
backgroundColor: CustomTheme.backgroundColor,
title: const Text(
"Create new group",
style: const TextStyle(fontSize: 20, fontWeight: FontWeight.bold),
),
centerTitle: true,
),
body: Column(
mainAxisAlignment: MainAxisAlignment.start,
children: [
Container(
margin: const EdgeInsets.symmetric(horizontal: 12, vertical: 10),
child: TextField(
decoration: InputDecoration(
filled: true,
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:17:35 +00:00
Outdated
Review
Copy Link

Single Quotes

Single Quotes
fillColor: CustomTheme.boxColor,
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:17:53 +00:00
Outdated
Review
Copy Link

const unnötig

`const` unnötig
hint: Text("Group name", style: TextStyle(fontSize: 18)),
enabledBorder: OutlineInputBorder(
borderRadius: BorderRadius.all(Radius.circular(12)),
borderSide: BorderSide(color: CustomTheme.boxBorder),
),
focusedBorder: OutlineInputBorder(
borderRadius: BorderRadius.all(Radius.circular(12)),
borderSide: BorderSide(color: CustomTheme.boxBorder),
),
floatingLabelBehavior: FloatingLabelBehavior.never,
),
),
),
Expanded(
child: Container(
margin: const EdgeInsets.symmetric(
horizontal: 12,
vertical: 10,
),
padding: const EdgeInsets.symmetric(
vertical: 10,
horizontal: 10,
),
decoration: BoxDecoration(
color: CustomTheme.boxColor,
border: Border.all(color: CustomTheme.boxBorder),
borderRadius: BorderRadius.circular(12),
),
child: Column(
crossAxisAlignment: CrossAxisAlignment.start,
children: [
SearchBar(
constraints: BoxConstraints(maxHeight: 45, minHeight: 45),
hintText: "Search for players",
hintStyle: WidgetStateProperty.all(
TextStyle(fontSize: 16),
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:18:37 +00:00
Outdated
Review
Copy Link

constraints: const BoxConstraints(...

`constraints: const BoxConstraints(...`
leading: Icon(Icons.search),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:18:43 +00:00
Outdated
Review
Copy Link

single quotes

single quotes
backgroundColor: WidgetStateProperty.all(
CustomTheme.boxColor,
),
side: WidgetStateProperty.all(
BorderSide(color: CustomTheme.boxBorder),
),
shape: WidgetStateProperty.all(
RoundedRectangleBorder(
borderRadius: BorderRadius.circular(12),
),
),
),
SizedBox(height: 10),
Text(
"Ausgewählte Spieler: (X)",
style: TextStyle(
fontSize: 16,
fontWeight: FontWeight.bold,
),
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:01 +00:00
Outdated
Review
Copy Link

const SizedBox()

`const SizedBox()`
SizedBox(height: 10),
Wrap(
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:11 +00:00
Outdated
Review
Copy Link

single quotes

single quotes
alignment: WrapAlignment.start,
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:21 +00:00
Outdated
Review
Copy Link

const TextStyle()

`const TextStyle()`
crossAxisAlignment: WrapCrossAlignment.start,
spacing: 8.0,
runSpacing: 8.0,
children: <Widget>[
for (var player in selectedPlayers)
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:31 +00:00
Outdated
Review
Copy Link

const SizedBox()

`const SizedBox()`
Container(
padding: EdgeInsets.all(5),
decoration: BoxDecoration(
color: CustomTheme.onBoxColor,
borderRadius: BorderRadius.circular(12),
),
child: Row(
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:33:12 +00:00
Outdated
Review
Copy Link

lieber var player in selectedPlayers?

lieber `var player in selectedPlayers`?
mainAxisAlignment: MainAxisAlignment.spaceBetween,
mainAxisSize: MainAxisSize.min,
children: [
SizedBox(width: 12),
Text(
player.name,
style: const TextStyle(
fontSize: 14,
fontWeight: FontWeight.w500,
),
),
SizedBox(width: 3),
GestureDetector(
child: const Icon(Icons.close, size: 20),
onTap: () {
setState(() {
selectedPlayers.remove(player);
});
},
),
],
),
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:49 +00:00
Outdated
Review
Copy Link

const SizedBox()

`const SizedBox()`
],
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:19:56 +00:00
Outdated
Review
Copy Link

single quotes

single quotes
SizedBox(height: 10),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:20:05 +00:00
Outdated
Review
Copy Link

const TextStyle

`const TextStyle`
Text(
"Alle Spieler:",
style: TextStyle(
fontSize: 16,
fontWeight: FontWeight.bold,
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:20:30 +00:00
Outdated
Review
Copy Link

const SizedBox()

`const SizedBox()`
),
),
SizedBox(height: 10),
FutureBuilder(
future: _allPlayersFuture,
builder:
(
BuildContext context,
AsyncSnapshot<List<Player>> snapshot,
) {
if (snapshot.hasError) {
return const Center(
child: TopCenteredMessage(
icon: Icons.report,
title: 'Error',
message: 'Player data couldn\'t\nbe loaded.',
),
);
}
if (snapshot.connectionState ==
ConnectionState.done &&
(!snapshot.hasData || snapshot.data!.isEmpty)) {
return const Center(
child: TopCenteredMessage(
icon: Icons.info,
title: 'Info',
message: 'No players created yet.',
),
);
}
final bool isLoading =
snapshot.connectionState ==
ConnectionState.waiting;
final List<Player> players = isLoading
? skeletonData
: (snapshot.data ?? []);
return Expanded(
child: Skeletonizer(
effect: PulseEffect(
from: Colors.grey[800]!,
to: Colors.grey[600]!,
duration: const Duration(milliseconds: 800),
),
enabled: isLoading,
enableSwitchAnimation: true,
switchAnimationConfig:
const SwitchAnimationConfig(
duration: Duration(milliseconds: 200),
switchInCurve: Curves.linear,
switchOutCurve: Curves.linear,
transitionBuilder: AnimatedSwitcher
.defaultTransitionBuilder,
layoutBuilder:
AnimatedSwitcher.defaultLayoutBuilder,
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:21:14 +00:00
Outdated
Review
Copy Link

Lieber isNotEmpty

Lieber `isNotEmpty`
child: ListView.builder(
itemCount: players.length,
itemBuilder:
(BuildContext context, int index) {
return Container(
margin: const EdgeInsets.symmetric(
horizontal: 5,
vertical: 5,
),
padding: const EdgeInsets.symmetric(
horizontal: 10,
),
decoration: BoxDecoration(
color: CustomTheme.boxColor,
border: Border.all(
color: CustomTheme.boxBorder,
),
borderRadius: BorderRadius.circular(
12,
),
),
child: Row(
mainAxisAlignment:
MainAxisAlignment.spaceBetween,
mainAxisSize: MainAxisSize.max,
children: [
Text(
players[index].name,
style: TextStyle(
fontSize: 16,
fontWeight: FontWeight.w500,
),
),
IconButton(
icon: Icon(Icons.add, size: 20),
onPressed: () {},
),
],
),
); //GroupTile(group: groups[index]);
},
),
),
);
},
),
],
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:21:28 +00:00
Outdated
Review
Copy Link

single quotes

single quotes
),
),
FullWidthButton(
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:42:30 +00:00
Outdated
Review
Copy Link

Button sieht deaktiviert aus wie ein SecondaryButton, styling anpassen. Ggf. Attribut festlegen, welches festlegt ob Button primary styling oder secondary styling hat

Button sieht deaktiviert aus wie ein SecondaryButton, styling anpassen. Ggf. Attribut festlegen, welches festlegt ob Button primary styling oder secondary styling hat
text: "Create group",
infillColor: CustomTheme.primaryColor,
borderColor: CustomTheme.primaryColor,
sizeRelativeToWidth: 0.95,
onPressed: () {},
),
SizedBox(height: 10),
FullWidthButton(
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:00 +00:00
Outdated
Review
Copy Link

const bzw #35 reinmergen und weglassen

`const` bzw #35 reinmergen und weglassen
text: "Cancel",
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 23:58:07 +00:00
Outdated
Review
Copy Link

Füg hier folgende Zeile ein und entferne die anderen beiden ifs

if (!context.mounted) return;
Füg hier folgende Zeile ein und entferne die anderen beiden `if`s ```dart if (!context.mounted) return; ```
sneeex commented 2025-11-19 00:05:02 +00:00
Outdated
Review
Copy Link

Füg hier folgende Zeile ein und entferne die anderen beiden ifs

if (!context.mounted) return;

Aber wenn's nicht mounted ist, will ich doch trotzdem noch die Felder clearen?

> Füg hier folgende Zeile ein und entferne die anderen beiden `if`s > ```dart > if (!context.mounted) return; > ``` Aber wenn's nicht mounted ist, will ich doch trotzdem noch die Felder clearen?
flixcoo commented 2025-11-19 00:18:47 +00:00
Outdated
Review
Copy Link

Der Context ist nicht mehr verfügbar, wenn z.B. das Widget garnicht mehr im widget tree ist. Also ist das dann eh egal

Der Context ist nicht mehr verfügbar, wenn z.B. das Widget garnicht mehr im widget tree ist. Also ist das dann eh egal
infillColor: CustomTheme.boxColor,
borderColor: CustomTheme.primaryColor,
sizeRelativeToWidth: 0.95,
onPressed: () {
Navigator.pop(context);
},
),
SizedBox(height: 20),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:27 +00:00
Outdated
Review
Copy Link

Bei asynchronen aufgaben mounted check

Bei asynchronen aufgaben `mounted` check
],
),
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:36 +00:00
Outdated
Review
Copy Link

mounted check

`mounted` check
),
);
}
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:47 +00:00
Outdated
Review
Copy Link

const

`const`
Future<void> addSamplePlayers(BuildContext context) async {
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:52 +00:00
Outdated
Review
Copy Link

single quotes

single quotes
final db = Provider.of<AppDatabase>(context, listen: false);
final playerCount = await db.playerDao.getPlayerCount();
if (playerCount == 0) {
for (int i = 1; i <= 10; i++) {
final player = Player(id: '$i', name: 'Spieler $i');
await db.playerDao.addPlayer(player: player);
}
print("10 Beispiel-Spieler wurden zur Datenbank hinzugefügt.");
final players = await db.playerDao.getAllPlayers();
for (int i = 0; i < players.length; i++) {
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:23:02 +00:00
Outdated
Review
Copy Link

const

`const`
print(players[i]);
}
}
}
}