remove todo

lib/presentation/views/main_menu/custom_navigation_bar.dart

@@ -1,12 +1,9 @@
 import 'package:flutter/material.dart';
 import 'package:game_tracker/core/custom_theme.dart';
-import 'package:game_tracker/data/dto/game.dart';
-import 'package:game_tracker/data/dto/group.dart';
-import 'package:game_tracker/data/dto/player.dart';
 import 'package:game_tracker/presentation/views/main_menu/game_history_view.dart';
-import 'package:game_tracker/presentation/views/main_menu/game_result_view.dart';
 import 'package:game_tracker/presentation/views/main_menu/groups_view.dart';
 import 'package:game_tracker/presentation/views/main_menu/home_view.dart';
+import 'package:game_tracker/presentation/views/main_menu/settings_view.dart';
 import 'package:game_tracker/presentation/views/main_menu/statistics_view.dart';
 import 'package:game_tracker/presentation/widgets/navbar_item.dart';
 
@@ -59,26 +56,7 @@ class _CustomNavigationBarState extends State<CustomNavigationBar>
             onPressed: () async {
               await Navigator.push(
                 context,
-                MaterialPageRoute(
+                MaterialPageRoute(builder: (_) => const SettingsView()),
-                  builder: (_) => GameResultView(
-                    game: Game(
-                      name: 'Test Game',
-                      players: [
-                        Player(name: 'Petrus'),
-                        Player(name: 'Peter'),
-                        Player(name: 'Petra'),
-                      ],
-                      group: Group(
-                        name: 'Die Petris',
-                        members: [
-                          Player(name: 'Petralia'),
-                          Player(name: 'Petrenlia'),
-                          Player(name: 'Petrumlia'),
-                        ],
-                      ),
-                    ),
-                  ),
-                ), //TODO Replace with Settingsview
               );
               setState(() {
                 tabKeyCount++;

lib/presentation/views/main_menu/game_history_view.dart

@@ -5,6 +5,7 @@ import 'package:game_tracker/data/dto/game.dart';
 import 'package:game_tracker/data/dto/group.dart';
 import 'package:game_tracker/data/dto/player.dart';
 import 'package:game_tracker/presentation/views/main_menu/create_group_view.dart';
+import 'package:game_tracker/presentation/views/main_menu/game_result_view.dart';
 import 'package:game_tracker/presentation/widgets/app_skeleton.dart';
 import 'package:game_tracker/presentation/widgets/buttons/custom_width_button.dart';
 import 'package:game_tracker/presentation/widgets/tiles/game_history_tile.dart';
@@ -21,7 +22,6 @@ class GameHistoryView extends StatefulWidget {
 class _GameHistoryViewState extends State<GameHistoryView> {
   late Future<List<Game>> _gameListFuture;
   late final AppDatabase db;
-  late bool isLoading = true;
 
   late final List<Game> skeletonData = List.filled(
     4,
@@ -50,9 +50,6 @@ class _GameHistoryViewState extends State<GameHistoryView> {
 
     Future.wait([_gameListFuture]).then((result) async {
       await Future.delayed(const Duration(milliseconds: 250));
-      setState(() {
-        isLoading = false;
-      });
     });
   }
 
@@ -86,14 +83,14 @@ class _GameHistoryViewState extends State<GameHistoryView> {
                       ),
                     );
                   }
-
+                  final bool isLoading =
+                      snapshot.connectionState == ConnectionState.waiting;
                   final List<Game> games =
                       (isLoading ? skeletonData : (snapshot.data ?? [])
                             ..sort(
                               (a, b) => b.createdAt.compareTo(a.createdAt),
                             ))
                           .toList();
-
                   return AppSkeleton(
                     enabled: isLoading,
                     child: ListView.builder(
@@ -106,8 +103,20 @@ class _GameHistoryViewState extends State<GameHistoryView> {
                           );
                         }
                         return GameHistoryTile(
+                          onTap: () async {
+                            await Navigator.push(
+                              context,
+                              MaterialPageRoute(
+                                builder: (context) =>
+                                    GameResultView(game: games[index]),
+                              ),
+                            );
+                            setState(() {
+                              _gameListFuture = db.gameDao.getAllGames();
+                            });
+                          },
                           game: games[index],
-                        ); // Placeholder
+                        );
                       },
                     ),
                   );

lib/presentation/views/main_menu/game_result_view.dart

@@ -1,10 +1,12 @@
 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/game.dart';
 import 'package:game_tracker/data/dto/player.dart';
 import 'package:game_tracker/presentation/widgets/buttons/custom_width_button.dart';
 import 'package:game_tracker/presentation/widgets/tiles/custom_radio_list_tile.dart';
 import 'package:game_tracker/presentation/widgets/top_centered_message.dart';
+import 'package:provider/provider.dart';
 
 class GameResultView extends StatefulWidget {
   final Game game;
@@ -17,11 +19,16 @@ class GameResultView extends StatefulWidget {
 
 class _GameResultViewState extends State<GameResultView> {
   late final List<Player> allPlayers;
-  Player? _player; //TODO: Set last winner as selected
+  late final AppDatabase db;
+  Player? _player;
 
   @override
   void initState() {
+    db = Provider.of<AppDatabase>(context, listen: false);
     allPlayers = getAllPlayers(widget.game);
+    if (widget.game.winner != null) {
+      _player = allPlayers.firstWhere((p) => p.id == widget.game.winner!.id);
+    }
     super.initState();
   }
 
@@ -112,11 +119,29 @@ class _GameResultViewState extends State<GameResultView> {
               text: 'Save',
               sizeRelativeToWidth: 0.95,
               onPressed: _player != null
-                  ? () {
+                  ? () async {
-                      print(
+                      print('Selected Winner: ${_player!.name}');
-                        'Selected Winner: ${_player!.name}',
+                      bool success = await db.gameDao.setWinner(
-                      ); //TODO: Add winner to db
+                        gameId: widget.game.id,
+                        winnerId: _player!.id,
+                      );
+                      if (!context.mounted) return;
+                      if (success) {
                         Navigator.pop(context);
+                      } else {
+                        ScaffoldMessenger.of(context).showSnackBar(
+                          SnackBar(
+                            backgroundColor: CustomTheme.boxColor,
+                            content: const Center(
+                              child: Text(
+                                'Error while setting winner, please try again',
+                                style: TextStyle(color: Colors.white),
+                              ),
+                            ),
+                          ),
+                        );
+                      }
+                      setState(() {});
                     }
                   : null,
             ),

lib/presentation/widgets/tiles/game_history_tile.dart

@@ -6,25 +6,24 @@ import 'package:intl/intl.dart';
 
 class GameHistoryTile extends StatefulWidget {
   final Game game;
+  final VoidCallback onTap;
 
-  const GameHistoryTile({
+  const GameHistoryTile({super.key, required this.game, required this.onTap});
-    super.key,
-    required this.game,
-  });
 
   @override
   State<GameHistoryTile> createState() => _GameHistoryTileState();
 }
 
 class _GameHistoryTileState extends State<GameHistoryTile> {
-
   @override
   Widget build(BuildContext context) {
     final group = widget.game.group;
     final winner = widget.game.winner;
     final allPlayers = _getAllPlayers();
 
-    return Container(
+    return GestureDetector(
+      onTap: widget.onTap,
+      child: Container(
         margin: const EdgeInsets.symmetric(horizontal: 12, vertical: 6),
         padding: const EdgeInsets.all(16),
         decoration: BoxDecoration(
@@ -50,10 +49,7 @@ class _GameHistoryTileState extends State<GameHistoryTile> {
                 ),
                 Text(
                   _formatDate(widget.game.createdAt),
-                style: const TextStyle(
+                  style: const TextStyle(fontSize: 12, color: Colors.grey),
-                  fontSize: 12,
-                  color: Colors.grey,
-                ),
                 ),
               ],
             ),
@@ -63,19 +59,12 @@ class _GameHistoryTileState extends State<GameHistoryTile> {
             if (group != null) ...[
               Row(
                 children: [
-                const Icon(
+                  const Icon(Icons.group, size: 16, color: Colors.grey),
-                  Icons.group,
-                  size: 16,
-                  color: Colors.grey,
-                ),
                   const SizedBox(width: 6),
                   Expanded(
                     child: Text(
                       group.name,
-                    style: const TextStyle(
+                      style: const TextStyle(fontSize: 14, color: Colors.grey),
-                      fontSize: 14,
-                      color: Colors.grey,
-                    ),
                       overflow: TextOverflow.ellipsis,
                     ),
                   ),
@@ -86,7 +75,10 @@ class _GameHistoryTileState extends State<GameHistoryTile> {
 
             if (winner != null) ...[
               Container(
-              padding: const EdgeInsets.symmetric(vertical: 8, horizontal: 12),
+                padding: const EdgeInsets.symmetric(
+                  vertical: 8,
+                  horizontal: 12,
+                ),
                 decoration: BoxDecoration(
                   color: Colors.green.withValues(alpha: 0.1),
                   borderRadius: BorderRadius.circular(8),
@@ -134,15 +126,13 @@ class _GameHistoryTileState extends State<GameHistoryTile> {
                 spacing: 6,
                 runSpacing: 6,
                 children: allPlayers.map((player) {
-                return TextIconTile(
+                  return TextIconTile(text: player.name, iconEnabled: false);
-                  text: player.name,
-                  iconEnabled: false,
-                );
                 }).toList(),
               ),
             ],
           ],
         ),
+      ),
     );
   }
 
@@ -187,5 +177,4 @@ class _GameHistoryTileState extends State<GameHistoryTile> {
 
     return allPlayers;
   }
-
 }