Merge pull request 'Anzeigefehler mit Snackbar' (#166) from bug/155-anzeigefehler-mit-snackbar into development

Reviewed-on: #166
Reviewed-by: gelbeinhalb <spam@yannick-weigert.de>

lib/presentation/views/main_menu/settings_view/settings_view.dart

@@ -37,7 +37,9 @@ class _SettingsViewState extends State<SettingsView> {
   Widget build(BuildContext context) {
     final loc = AppLocalizations.of(context);
     return ScaffoldMessenger(
-      child: Scaffold(
+      child: Builder(
+        builder: (scaffoldMessengerContext) {
+          return Scaffold(
             appBar: AppBar(backgroundColor: CustomTheme.backgroundColor),
             backgroundColor: CustomTheme.backgroundColor,
             body: SingleChildScrollView(
@@ -57,7 +59,11 @@ class _SettingsViewState extends State<SettingsView> {
                     ),
                   ),
                   Padding(
-                padding: const EdgeInsets.only(left: 16, top: 10, bottom: 10),
+                    padding: const EdgeInsets.only(
+                      left: 16,
+                      top: 10,
+                      bottom: 10,
+                    ),
                     child: Text(
                       textAlign: TextAlign.start,
                       loc.data,
@@ -73,13 +79,18 @@ class _SettingsViewState extends State<SettingsView> {
                     suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
                     onPressed: () async {
                       final String json =
-                      await DataTransferService.getAppDataAsJson(context);
+                          await DataTransferService.getAppDataAsJson(
+                            scaffoldMessengerContext,
+                          );
                       final result = await DataTransferService.exportData(
                         json,
                         'game_tracker-data',
                       );
-                  if (!context.mounted) return;
+                      if (!scaffoldMessengerContext.mounted) return;
-                  showExportSnackBar(context: context, result: result);
+                      showExportSnackBar(
+                        context: scaffoldMessengerContext,
+                        result: result,
+                      );
                     },
                   ),
                   SettingsListTile(
@@ -87,9 +98,14 @@ class _SettingsViewState extends State<SettingsView> {
                     icon: Icons.download,
                     suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
                     onPressed: () async {
-                  final result = await DataTransferService.importData(context);
+                      final result = await DataTransferService.importData(
-                  if (!context.mounted) return;
+                        scaffoldMessengerContext,
-                  showImportSnackBar(context: context, result: result);
+                      );
+                      if (!scaffoldMessengerContext.mounted) return;
+                      showImportSnackBar(
+                        context: scaffoldMessengerContext,
+                        result: result,
+                      );
                     },
                   ),
                   SettingsListTile(
@@ -98,28 +114,33 @@ class _SettingsViewState extends State<SettingsView> {
                     suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
                     onPressed: () {
                       showDialog<bool>(
-                    context: context,
+                        context: scaffoldMessengerContext,
-                    builder: (context) => AlertDialog(
+                        builder: (dialogContext) => AlertDialog(
                           title: Text('${loc.delete_all_data}?'),
                           content: Text(loc.this_cannot_be_undone),
                           actions: [
                             TextButton(
-                          onPressed: () => Navigator.of(context).pop(false),
+                              onPressed: () =>
+                                  Navigator.of(dialogContext).pop(false),
                               child: Text(loc.cancel),
                             ),
                             TextButton(
-                          onPressed: () => Navigator.of(context).pop(true),
+                              onPressed: () =>
+                                  Navigator.of(dialogContext).pop(true),
                               child: Text(loc.delete),
                             ),
                           ],
                         ),
                       ).then((confirmed) {
-                    if (confirmed == true && context.mounted) {
+                        if (confirmed == true &&
-                      DataTransferService.deleteAllData(context);
+                            scaffoldMessengerContext.mounted) {
+                          DataTransferService.deleteAllData(
+                            scaffoldMessengerContext,
+                          );
                           showSnackbar(
-                        context: context,
+                            context: scaffoldMessengerContext,
                             message: AppLocalizations.of(
-                          context,
+                              scaffoldMessengerContext,
                             ).data_successfully_deleted,
                           );
                         }
@@ -127,7 +148,11 @@ class _SettingsViewState extends State<SettingsView> {
                     },
                   ),
                   Padding(
-                padding: const EdgeInsets.only(left: 16, top: 10, bottom: 10),
+                    padding: const EdgeInsets.only(
+                      left: 16,
+                      top: 10,
+                      bottom: 10,
+                    ),
                     child: Text(
                       textAlign: TextAlign.start,
                       loc.legal,
@@ -176,7 +201,9 @@ class _SettingsViewState extends State<SettingsView> {
                                 GestureDetector(
                                   child: const Icon(Icons.language),
                                   onTap: () => {
-                                launchUrl(Uri.parse('https://liquid-dev.de')),
+                                    launchUrl(
+                                      Uri.parse('https://liquid-dev.de'),
+                                    ),
                                   },
                                 ),
                                 GestureDetector(
@@ -225,6 +252,8 @@ class _SettingsViewState extends State<SettingsView> {
                 ],
               ),
             ),
+          );
+        },
       ),
     );
   }
@@ -285,10 +314,11 @@ class _SettingsViewState extends State<SettingsView> {
     Duration duration = const Duration(seconds: 3),
     VoidCallback? action,
   }) {
+    if (!context.mounted) return;
+
     final loc = AppLocalizations.of(context);
-    final messenger = ScaffoldMessenger.of(context);
+    ScaffoldMessenger.of(context).hideCurrentSnackBar();
-    messenger.hideCurrentSnackBar();
+    ScaffoldMessenger.of(context).showSnackBar(
-    messenger.showSnackBar(
       SnackBar(
         content: Text(message, style: const TextStyle(color: Colors.white)),
         backgroundColor: CustomTheme.onBoxColor,

lib/presentation/widgets/player_selection.dart

@@ -181,6 +181,7 @@ class _PlayerSelectionState extends State<PlayerSelection> {
                   icon: Icons.info,
                   title: loc.info,
                   message: _getInfoText(context),
+                  fullscreen: false,
                 ),
                 child: ListView.builder(
                   itemCount: suggestedPlayers.length,
@@ -294,6 +295,7 @@ class _PlayerSelectionState extends State<PlayerSelection> {
   /// [message] - The message to display in the snackbar.
   void showSnackBarMessage(String message) {
     if (!context.mounted) return;
+    ScaffoldMessenger.of(context).hideCurrentSnackBar();
     ScaffoldMessenger.of(context).showSnackBar(
       SnackBar(
         backgroundColor: CustomTheme.boxColor,

lib/presentation/widgets/top_centered_message.dart

@@ -10,6 +10,7 @@ class TopCenteredMessage extends StatelessWidget {
     required this.icon,
     required this.title,
     required this.message,
+    this.fullscreen = true,
   });
 
   /// The icon to display above the title.
@@ -21,13 +22,18 @@ class TopCenteredMessage extends StatelessWidget {
   /// The message text to display below the title.
   final String message;
 
+  final bool fullscreen;
+
   @override
   Widget build(BuildContext context) {
     return Container(
-      padding: const EdgeInsets.only(top: 100),
+      padding: fullscreen ? const EdgeInsets.only(top: 100) : null,
       margin: const EdgeInsets.symmetric(horizontal: 10),
       alignment: Alignment.topCenter,
       child: Column(
+        mainAxisAlignment: fullscreen
+            ? MainAxisAlignment.start
+            : MainAxisAlignment.center,
         children: [
           Icon(icon, size: 45),
           const SizedBox(height: 10),

pubspec.yaml

@@ -1,7 +1,7 @@
 name: game_tracker
 description: "Game Tracking App for Card Games"
 publish_to: 'none'
-version: 0.0.6+208
+version: 0.0.6+209
 
 environment:
   sdk: ^3.8.1