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>
Added hiding to prevent stacking snackbars
2026-01-13 21:09:21 +00:00 · 2026-01-13 21:45:22 +01:00 · 2026-01-13 21:44:38 +01:00 · 2026-01-13 14:30:53 +00:00 · 2026-01-13 15:25:59 +01:00 · 2026-01-12 23:19:37 +01:00
4 changed files with 222 additions and 184 deletions
--- a/lib/presentation/views/main_menu/settings_view/settings_view.dart
+++ b/lib/presentation/views/main_menu/settings_view/settings_view.dart
@@ -37,194 +37,223 @@ class _SettingsViewState extends State<SettingsView> {
  Widget build(BuildContext context) {
    final loc = AppLocalizations.of(context);
    return ScaffoldMessenger(
-      child: Scaffold(
-        appBar: AppBar(backgroundColor: CustomTheme.backgroundColor),
-        backgroundColor: CustomTheme.backgroundColor,
-        body: SingleChildScrollView(
-          child: Column(
-            mainAxisAlignment: MainAxisAlignment.start,
-            crossAxisAlignment: CrossAxisAlignment.start,
-            children: [
-              Padding(
-                padding: const EdgeInsets.only(left: 16, bottom: 10),
-                child: Text(
-                  textAlign: TextAlign.start,
-                  loc.settings,
-                  style: const TextStyle(
-                    fontSize: 28,
-                    fontWeight: FontWeight.bold,
-                  ),
-                ),
-              ),
-              Padding(
-                padding: const EdgeInsets.only(left: 16, top: 10, bottom: 10),
-                child: Text(
-                  textAlign: TextAlign.start,
-                  loc.data,
-                  style: const TextStyle(
-                    fontSize: 22,
-                    fontWeight: FontWeight.bold,
-                  ),
-                ),
-              ),
-              SettingsListTile(
-                title: loc.export_data,
-                icon: Icons.upload,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: () async {
-                  final String json =
-                      await DataTransferService.getAppDataAsJson(context);
-                  final result = await DataTransferService.exportData(
-                    json,
-                    'game_tracker-data',
-                  );
-                  if (!context.mounted) return;
-                  showExportSnackBar(context: context, result: result);
-                },
-              ),
-              SettingsListTile(
-                title: loc.import_data,
-                icon: Icons.download,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: () async {
-                  final result = await DataTransferService.importData(context);
-                  if (!context.mounted) return;
-                  showImportSnackBar(context: context, result: result);
-                },
-              ),
-              SettingsListTile(
-                title: loc.delete_all_data,
-                icon: Icons.delete,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: () {
-                  showDialog<bool>(
-                    context: context,
-                    builder: (context) => AlertDialog(
-                      title: Text('${loc.delete_all_data}?'),
-                      content: Text(loc.this_cannot_be_undone),
-                      actions: [
-                        TextButton(
-                          onPressed: () => Navigator.of(context).pop(false),
-                          child: Text(loc.cancel),
-                        ),
-                        TextButton(
-                          onPressed: () => Navigator.of(context).pop(true),
-                          child: Text(loc.delete),
-                        ),
-                      ],
+      child: Builder(
+        builder: (scaffoldMessengerContext) {
+          return Scaffold(
+            appBar: AppBar(backgroundColor: CustomTheme.backgroundColor),
+            backgroundColor: CustomTheme.backgroundColor,
+            body: SingleChildScrollView(
+              child: Column(
+                mainAxisAlignment: MainAxisAlignment.start,
+                crossAxisAlignment: CrossAxisAlignment.start,
+                children: [
+                  Padding(
+                    padding: const EdgeInsets.only(left: 16, bottom: 10),
+                    child: Text(
+                      textAlign: TextAlign.start,
+                      loc.settings,
+                      style: const TextStyle(
+                        fontSize: 28,
+                        fontWeight: FontWeight.bold,
+                      ),
                    ),
-                  ).then((confirmed) {
-                    if (confirmed == true && context.mounted) {
-                      DataTransferService.deleteAllData(context);
-                      showSnackbar(
-                        context: context,
-                        message: AppLocalizations.of(
-                          context,
-                        ).data_successfully_deleted,
+                  ),
+                  Padding(
+                    padding: const EdgeInsets.only(
+                      left: 16,
+                      top: 10,
+                      bottom: 10,
+                    ),
+                    child: Text(
+                      textAlign: TextAlign.start,
+                      loc.data,
+                      style: const TextStyle(
+                        fontSize: 22,
+                        fontWeight: FontWeight.bold,
+                      ),
+                    ),
+                  ),
+                  SettingsListTile(
+                    title: loc.export_data,
+                    icon: Icons.upload,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: () async {
+                      final String json =
+                          await DataTransferService.getAppDataAsJson(
+                            scaffoldMessengerContext,
+                          );
+                      final result = await DataTransferService.exportData(
+                        json,
+                        'game_tracker-data',
                      );
-                    }
-                  });
-                },
-              ),
-              Padding(
-                padding: const EdgeInsets.only(left: 16, top: 10, bottom: 10),
-                child: Text(
-                  textAlign: TextAlign.start,
-                  loc.legal,
-                  style: const TextStyle(
-                    fontSize: 22,
-                    fontWeight: FontWeight.bold,
+                      if (!scaffoldMessengerContext.mounted) return;
+                      showExportSnackBar(
+                        context: scaffoldMessengerContext,
+                        result: result,
+                      );
+                    },
                  ),
-                ),
-              ),
-              SettingsListTile(
-                title: loc.licenses,
-                icon: Icons.insert_drive_file,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: () {
-                  Navigator.of(context).push(
-                    MaterialPageRoute(
-                      builder: (context) => const LicensesView(),
-                    ),
-                  );
-                },
-              ),
-              SettingsListTile(
-                title: loc.legal_notice,
-                icon: Icons.account_balance_sharp,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: null,
-              ),
-              SettingsListTile(
-                title: loc.privacy_policy,
-                icon: Icons.gpp_good_rounded,
-                suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
-                onPressed: null,
-              ),
-              Padding(
-                padding: const EdgeInsets.only(top: 30, bottom: 20),
-                child: Center(
-                  child: Column(
-                    spacing: 4,
-                    children: [
-                      Padding(
-                        padding: const EdgeInsets.only(bottom: 12),
-                        child: Row(
-                          mainAxisAlignment: MainAxisAlignment.center,
-                          spacing: 40,
-                          children: [
-                            GestureDetector(
-                              child: const Icon(Icons.language),
-                              onTap: () => {
-                                launchUrl(Uri.parse('https://liquid-dev.de')),
-                              },
+                  SettingsListTile(
+                    title: loc.import_data,
+                    icon: Icons.download,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: () async {
+                      final result = await DataTransferService.importData(
+                        scaffoldMessengerContext,
+                      );
+                      if (!scaffoldMessengerContext.mounted) return;
+                      showImportSnackBar(
+                        context: scaffoldMessengerContext,
+                        result: result,
+                      );
+                    },
+                  ),
+                  SettingsListTile(
+                    title: loc.delete_all_data,
+                    icon: Icons.delete,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: () {
+                      showDialog<bool>(
+                        context: scaffoldMessengerContext,
+                        builder: (dialogContext) => AlertDialog(
+                          title: Text('${loc.delete_all_data}?'),
+                          content: Text(loc.this_cannot_be_undone),
+                          actions: [
+                            TextButton(
+                              onPressed: () =>
+                                  Navigator.of(dialogContext).pop(false),
+                              child: Text(loc.cancel),
                            ),
-                            GestureDetector(
-                              child: const FaIcon(FontAwesomeIcons.github),
-                              onTap: () => {
-                                launchUrl(
-                                  Uri.parse(
-                                    'https://github.com/liquiddevelopmentde',
-                                  ),
-                                ),
-                              },
-                            ),
-                            GestureDetector(
-                              child: Icon(
-                                Platform.isIOS
-                                    ? CupertinoIcons.mail_solid
-                                    : Icons.email,
-                              ),
-                              onTap: () => launchUrl(
-                                Uri.parse('mailto:hi@liquid-dev.de'),
-                              ),
+                            TextButton(
+                              onPressed: () =>
+                                  Navigator.of(dialogContext).pop(true),
+                              child: Text(loc.delete),
                            ),
                          ],
                        ),
-                      ),
-                      Text(
-                        '© ${DateFormat('yyyy').format(DateTime.now())} Liquid Development',
-                        style: TextStyle(
-                          color: Colors.grey.shade600,
-                          fontSize: 14,
-                          fontWeight: FontWeight.w600,
-                        ),
-                      ),
-                      Text(
-                        'Version ${_packageInfo.version} (${_packageInfo.buildNumber})',
-                        style: TextStyle(
-                          color: Colors.grey.shade600,
-                          fontSize: 14,
-                          fontWeight: FontWeight.w600,
-                        ),
-                      ),
-                    ],
+                      ).then((confirmed) {
+                        if (confirmed == true &&
+                            scaffoldMessengerContext.mounted) {
+                          DataTransferService.deleteAllData(
+                            scaffoldMessengerContext,
+                          );
+                          showSnackbar(
+                            context: scaffoldMessengerContext,
+                            message: AppLocalizations.of(
+                              scaffoldMessengerContext,
+                            ).data_successfully_deleted,
+                          );
+                        }
+                      });
+                    },
                  ),
-                ),
+                  Padding(
+                    padding: const EdgeInsets.only(
+                      left: 16,
+                      top: 10,
+                      bottom: 10,
+                    ),
+                    child: Text(
+                      textAlign: TextAlign.start,
+                      loc.legal,
+                      style: const TextStyle(
+                        fontSize: 22,
+                        fontWeight: FontWeight.bold,
+                      ),
+                    ),
+                  ),
+                  SettingsListTile(
+                    title: loc.licenses,
+                    icon: Icons.insert_drive_file,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: () {
+                      Navigator.of(context).push(
+                        MaterialPageRoute(
+                          builder: (context) => const LicensesView(),
+                        ),
+                      );
+                    },
+                  ),
+                  SettingsListTile(
+                    title: loc.legal_notice,
+                    icon: Icons.account_balance_sharp,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: null,
+                  ),
+                  SettingsListTile(
+                    title: loc.privacy_policy,
+                    icon: Icons.gpp_good_rounded,
+                    suffixWidget: const Icon(Icons.arrow_forward_ios, size: 16),
+                    onPressed: null,
+                  ),
+                  Padding(
+                    padding: const EdgeInsets.only(top: 30, bottom: 20),
+                    child: Center(
+                      child: Column(
+                        spacing: 4,
+                        children: [
+                          Padding(
+                            padding: const EdgeInsets.only(bottom: 12),
+                            child: Row(
+                              mainAxisAlignment: MainAxisAlignment.center,
+                              spacing: 40,
+                              children: [
+                                GestureDetector(
+                                  child: const Icon(Icons.language),
+                                  onTap: () => {
+                                    launchUrl(
+                                      Uri.parse('https://liquid-dev.de'),
+                                    ),
+                                  },
+                                ),
+                                GestureDetector(
+                                  child: const FaIcon(FontAwesomeIcons.github),
+                                  onTap: () => {
+                                    launchUrl(
+                                      Uri.parse(
+                                        'https://github.com/liquiddevelopmentde',
+                                      ),
+                                    ),
+                                  },
+                                ),
+                                GestureDetector(
+                                  child: Icon(
+                                    Platform.isIOS
+                                        ? CupertinoIcons.mail_solid
+                                        : Icons.email,
+                                  ),
+                                  onTap: () => launchUrl(
+                                    Uri.parse('mailto:hi@liquid-dev.de'),
+                                  ),
+                                ),
+                              ],
+                            ),
+                          ),
+                          Text(
+                            '© ${DateFormat('yyyy').format(DateTime.now())} Liquid Development',
+                            style: TextStyle(
+                              color: Colors.grey.shade600,
+                              fontSize: 14,
+                              fontWeight: FontWeight.w600,
+                            ),
+                          ),
+                          Text(
+                            'Version ${_packageInfo.version} (${_packageInfo.buildNumber})',
+                            style: TextStyle(
+                              color: Colors.grey.shade600,
+                              fontSize: 14,
+                              fontWeight: FontWeight.w600,
+                            ),
+                          ),
+                        ],
+                      ),
+                    ),
+                  ),
+                ],
              ),
-            ],
-          ),
-        ),
+            ),
+          );
+        },
      ),
    );
  }
@@ -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);
-    messenger.hideCurrentSnackBar();
-    messenger.showSnackBar(
+    ScaffoldMessenger.of(context).hideCurrentSnackBar();
+    ScaffoldMessenger.of(context).showSnackBar(
      SnackBar(
        content: Text(message, style: const TextStyle(color: Colors.white)),
        backgroundColor: CustomTheme.onBoxColor,
--- a/lib/presentation/widgets/player_selection.dart
+++ b/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,
--- a/lib/presentation/widgets/top_centered_message.dart
+++ b/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),
--- a/pubspec.yaml
+++ b/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
Author	SHA1	Message	Date
Felix Kirchner	e483fc38f3	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>	2026-01-13 21:09:21 +00:00
Felix Kirchner	7be0b96491	Added hiding to prevent stacking snackbars All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m19s Details Pull Request Pipeline / lint (pull_request) Successful in 2m23s Details	2026-01-13 21:45:22 +01:00
Felix Kirchner	efdb5e0361	Fixed snackbar	2026-01-13 21:44:38 +01:00
Mathis Kirchner	7a1752f773	Merge pull request 'TopCenteredMessage Anzeigefehler' (#161 ) from bug/159-topcenteredmessage-anzeigefehler into development Reviewed-on: #161 Reviewed-by: Mathis Kirchner <mathis.kirchner.mk@gmail.com>	2026-01-13 14:30:53 +00:00
Felix Kirchner	57357f8aad	Merge branch 'development' into bug/159-topcenteredmessage-anzeigefehler All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m3s Details Pull Request Pipeline / lint (pull_request) Successful in 2m12s Details # Conflicts: # pubspec.yaml	2026-01-13 15:25:59 +01:00
Felix Kirchner	d5bd0bca5f	Fixed displayment bug with TopCenteredMessage All checks were successful Pull Request Pipeline / test (pull_request) Successful in 2m6s Details Pull Request Pipeline / lint (pull_request) Successful in 2m11s Details	2026-01-12 23:19:37 +01:00