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

241
lib/presentation/views/main_menu/create_group/create_group_view.dart
View File

@@ -36,8 +36,8 @@ class _CreateGroupViewState extends State<CreateGroupView> {
_allPlayersFuture.then((loadedPlayers) { _allPlayersFuture.then((loadedPlayers) {
setState(() { setState(() {
loadedPlayers.sort((a, b) => a.name.compareTo(b.name)); loadedPlayers.sort((a, b) => a.name.compareTo(b.name));
allPlayers = loadedPlayers; allPlayers = [...loadedPlayers];
suggestedPlayers = loadedPlayers; suggestedPlayers = [...loadedPlayers];
}); });
}); });
} }
@@ -123,7 +123,7 @@ class _CreateGroupViewState extends State<CreateGroupView> {
onChanged: (value) { onChanged: (value) {
setState(() { setState(() {
if (value.isEmpty) { if (value.isEmpty) {
suggestedPlayers = allPlayers; suggestedPlayers = [...allPlayers];
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()`
} else { } else {
suggestedPlayers = allPlayers.where((player) { suggestedPlayers = allPlayers.where((player) {
return player.name.toLowerCase().contains( return player.name.toLowerCase().contains(
@@ -197,131 +197,122 @@ class _CreateGroupViewState extends State<CreateGroupView> {
SizedBox(height: 10), SizedBox(height: 10),
FutureBuilder( FutureBuilder(
future: _allPlayersFuture, future: _allPlayersFuture,
builder: builder: (BuildContext context, AsyncSnapshot<List<Player>> snapshot) {
( if (snapshot.hasError) {
BuildContext context, return const Center(
AsyncSnapshot<List<Player>> snapshot, child: TopCenteredMessage(
) { icon: Icons.report,
if (snapshot.hasError) { title: 'Error',
return const Center( message: 'Player data couldn\'t\nbe loaded.',
child: TopCenteredMessage( ),
icon: Icons.report, );
title: 'Error', }
message: 'Player data couldn\'t\nbe loaded.', if (snapshot.connectionState == ConnectionState.done &&
), (!snapshot.hasData ||
); snapshot.data!.isEmpty ||
} (selectedPlayers.isEmpty &&
if (snapshot.connectionState == allPlayers.isEmpty))) {
ConnectionState.done && return const Center(
(!snapshot.hasData || child: TopCenteredMessage(
snapshot.data!.isEmpty || icon: Icons.info,
(suggestedPlayers.isEmpty && title: 'Info',
allPlayers.isEmpty))) { message: 'No players created yet.',
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`
return const Center( ),
child: TopCenteredMessage( );
icon: Icons.info, }
title: 'Info', final bool isLoading =
message: 'No players created yet.', snapshot.connectionState == ConnectionState.waiting;
), return Expanded(
); child: Skeletonizer(
} effect: PulseEffect(
final bool isLoading = from: Colors.grey[800]!,
snapshot.connectionState == to: Colors.grey[600]!,
ConnectionState.waiting; duration: const Duration(milliseconds: 800),
return Expanded( ),
child: Skeletonizer( enabled: isLoading,
effect: PulseEffect( enableSwitchAnimation: true,
from: Colors.grey[800]!, switchAnimationConfig: const SwitchAnimationConfig(
to: Colors.grey[600]!, duration: Duration(milliseconds: 200),
duration: const Duration(milliseconds: 800), switchInCurve: Curves.linear,
), switchOutCurve: Curves.linear,
enabled: isLoading, transitionBuilder:
enableSwitchAnimation: true, AnimatedSwitcher.defaultTransitionBuilder,
switchAnimationConfig: layoutBuilder:
const SwitchAnimationConfig( AnimatedSwitcher.defaultLayoutBuilder,
duration: Duration(milliseconds: 200), ),
switchInCurve: Curves.linear, child:
switchOutCurve: Curves.linear, (suggestedPlayers.isEmpty &&
transitionBuilder: AnimatedSwitcher !allPlayers.isEmpty)
.defaultTransitionBuilder, ? TopCenteredMessage(
layoutBuilder: icon: Icons.info,
AnimatedSwitcher.defaultLayoutBuilder, title: 'Info',
), message:
child: (selectedPlayers.length ==
(suggestedPlayers.isEmpty && allPlayers.length)
!allPlayers.isEmpty) ? 'No more players to add.'
? TopCenteredMessage( : 'No players found with that name.',
icon: Icons.info, )
title: 'Info', : ListView.builder(
message: itemCount: suggestedPlayers.length,
'No players found with that name.', itemBuilder: (BuildContext context, int index) {
) return Container(
: ListView.builder( margin: const EdgeInsets.symmetric(
itemCount: suggestedPlayers.length, horizontal: 5,
itemBuilder: (BuildContext context, int index) { vertical: 5,
return Container( ),
margin: const EdgeInsets.symmetric( padding: const EdgeInsets.symmetric(
horizontal: 5, horizontal: 10,
vertical: 5, ),
), decoration: BoxDecoration(
padding: const EdgeInsets.symmetric( color: CustomTheme.boxColor,
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
horizontal: 10, border: Border.all(
), color: CustomTheme.boxBorder,
decoration: BoxDecoration( ),
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
color: CustomTheme.boxColor, borderRadius: BorderRadius.circular(
border: Border.all( 12,
color: CustomTheme.boxBorder, ),
),
child: Row(
mainAxisAlignment:
MainAxisAlignment.spaceBetween,
mainAxisSize: MainAxisSize.max,
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
children: [
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
Text(
suggestedPlayers[index].name,
style: TextStyle(
fontSize: 16,
fontWeight: FontWeight.w500,
), ),
borderRadius:
BorderRadius.circular(12),
), ),
child: Row( IconButton(
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
mainAxisAlignment: icon: Icon(Icons.add, size: 20),
MainAxisAlignment onPressed: () {
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
.spaceBetween, setState(() {
mainAxisSize: MainAxisSize.max, if (!selectedPlayers.contains(
children: [ suggestedPlayers[index],
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:22:47 +00:00
Outdated
Review
Copy Link

const

`const`
Text( )) {
suggestedPlayers[index].name, selectedPlayers.add(
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
style: TextStyle( suggestedPlayers[index],
fontSize: 16, );
fontWeight: FontWeight.w500, selectedPlayers.sort(
), (a, b) => a.name
), .compareTo(b.name),
IconButton( );
icon: Icon( suggestedPlayers.remove(
Icons.add, suggestedPlayers[index],
size: 20, );
), }
sneeex marked this conversation as resolved Outdated
flixcoo commented 2025-11-18 19:23:02 +00:00
Outdated
Review
Copy Link

const

`const`
onPressed: () { });
setState(() { },
if (!selectedPlayers.contains(
suggestedPlayers[index],
)) {
selectedPlayers.add(
suggestedPlayers[index],
);
selectedPlayers.sort(
(a, b) =>
a.name.compareTo(
b.name,
),
);
suggestedPlayers.remove(
suggestedPlayers[index],
);
}
});
},
),
],
), ),
); ],
}, ),
), );
), },
); ),
}, ),
);
},
), ),
], ],
), ),