remove error msgs in offline mode

lichess-org · May 13, 2024 · 3845636 · 3845636
1 parent d0f4441
commit 3845636
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 20 deletions.
diff --git a/lib/src/view/puzzle/dashboard_screen.dart b/lib/src/view/puzzle/dashboard_screen.dart
@@ -9,6 +9,7 @@ import 'package:lichess_mobile/src/model/puzzle/puzzle.dart';
 import 'package:lichess_mobile/src/model/puzzle/puzzle_providers.dart';
 import 'package:lichess_mobile/src/model/puzzle/puzzle_theme.dart';
 import 'package:lichess_mobile/src/styles/styles.dart';
+import 'package:lichess_mobile/src/utils/connectivity.dart';
 import 'package:lichess_mobile/src/utils/l10n_context.dart';
 import 'package:lichess_mobile/src/utils/string.dart';
 import 'package:lichess_mobile/src/widgets/adaptive_choice_picker.dart';
@@ -123,25 +124,33 @@ class PuzzleDashboardWidget extends ConsumerWidget {
         );
       },
       error: (e, s) {
+        final connectivity = ref.watch(connectivityChangesProvider);
+        final isOnline = connectivity.value?.isOnline ?? true;
+
         debugPrint(
           'SEVERE: [PuzzleDashboardWidget] could not load puzzle dashboard; $e\n$s',
         );
-        return Padding(
-          padding: Styles.bodySectionPadding,
-          child: Column(
-            crossAxisAlignment: CrossAxisAlignment.start,
-            children: [
-              Text(
-                context.l10n.puzzlePuzzleDashboard,
-                style: Styles.sectionTitle,
-              ),
-              if (e is ClientException && e.message.contains('404'))
-                Text(context.l10n.puzzleNoPuzzlesToShow)
-              else
-                const Text('Could not load dashboard.'),
-            ],
-          ),
-        );
+
+        if (!isOnline) {
+          return Padding(
+            padding: Styles.bodySectionPadding,
+            child: Column(
+              crossAxisAlignment: CrossAxisAlignment.start,
+              children: [
+                Text(
+                  context.l10n.puzzlePuzzleDashboard,
+                  style: Styles.sectionTitle,
+                ),
+                if (e is ClientException && e.message.contains('404'))
+                  Text(context.l10n.puzzleNoPuzzlesToShow)
+                else
+                  const Text('Could not load dashboard.'),
+              ],
+            ),
+          );
+        } else {
+          return const SizedBox.shrink();
+        }
       },
       loading: () {
         final loaderHeight = MediaQuery.sizeOf(context).width;

diff --git a/lib/src/view/puzzle/puzzle_tab_screen.dart b/lib/src/view/puzzle/puzzle_tab_screen.dart
@@ -347,13 +347,21 @@ class PuzzleHistoryWidget extends ConsumerWidget {
         );
       },
       error: (e, s) {
+        final connectivity = ref.watch(connectivityChangesProvider);
+        final isOnline = connectivity.value?.isOnline ?? true;
+
         debugPrint(
           'SEVERE: [PuzzleHistoryWidget] could not load puzzle history',
         );
-        return Padding(
-          padding: Styles.bodySectionPadding,
-          child: const Text('Could not load Puzzle history.'),
-        );
+
+        if (!isOnline) {
+          return Padding(
+            padding: Styles.bodySectionPadding,
+            child: const Text('Could not load Puzzle history.'),
+          );
+        } else {
+          return const SizedBox.shrink();
+        }
       },
       loading: () => Shimmer(
         child: ShimmerLoading(