Merge pull request #141 from flixcoo/bug/139-cancel-button-triggers-confetti

Cancel button triggers confetti

lib/presentation/views/active_game_view.dart

@@ -24,11 +24,19 @@ class ActiveGameView extends StatefulWidget {
 }
 
 class _ActiveGameViewState extends State<ActiveGameView> {
+  /// Constant value to represent a press on the cancel button in round view.
+  static const int kRoundCancelled = -1;
+
   final confettiController = ConfettiController(
     duration: const Duration(seconds: 10),
   );
+
   late final GameSession gameSession;
+
+  /// A list of the ranks for each player corresponding to their index in sortedPlayerIndices
   late List<int> denseRanks;
+
+  /// A list of player indices sorted by their scores in ascending order.
   late List<int> sortedPlayerIndices;
 
   @override
@@ -453,6 +461,9 @@ class _ActiveGameViewState extends State<ActiveGameView> {
       ),
     );
 
+    // If the user presses the cancel button
+    if (round == kRoundCancelled) return;
+
     if (widget.gameSession.isGameFinished && context.mounted) {
       _playFinishAnimation(context);
     }

lib/presentation/views/round_view.dart

@@ -80,7 +80,7 @@ class _RoundViewState extends State<RoundView> {
             padding: EdgeInsets.zero,
             onPressed: () => {
               LocalStorageService.saveGameSessions(),
-              Navigator.pop(context)
+              Navigator.pop(context, -1)
             },
             child: Text(AppLocalizations.of(context).cancel),
           ),

pubspec.yaml

@@ -2,7 +2,7 @@ name: cabo_counter
 description: "Mobile app for the card game Cabo"
 publish_to: 'none'
 
-version: 0.5.3+594
+version: 0.5.3+595
 
 environment:
   sdk: ^3.5.4