🐛 Fix duplicate no connectivity dialog shown when app start with no c…

…onnection.
openjmu · Feb 20, 2020 · 3ef3d19 · 3ef3d19
1 parent 96c9fc8
commit 3ef3d19
Showing 1 changed file with 14 additions and 19 deletions.
diff --git a/lib/main.dart b/lib/main.dart
@@ -65,7 +65,9 @@ class OpenJMUAppState extends State<OpenJMUApp> with WidgetsBindingObserver {
     WidgetsBinding.instance.addObserver(this);
     tryRecoverLoginInfo();
 
-    Connectivity().checkConnectivity().then(checkIfNoConnectivity);
+    SchedulerBinding.instance.addPostFrameCallback((_) {
+      Connectivity().checkConnectivity().then(checkIfNoConnectivity);
+    });
     connectivitySubscription = Connectivity().onConnectivityChanged.listen(
       (ConnectivityResult result) {
         checkIfNoConnectivity(result);
@@ -170,29 +172,22 @@ class OpenJMUAppState extends State<OpenJMUApp> with WidgetsBindingObserver {
   }
 
   void checkIfNoConnectivity(ConnectivityResult result) {
-    if (result == ConnectivityResult.none) {
-      if (mounted) {
-        connectivityToastFuture = showToastWidget(
-          noConnectivityWidget,
-          duration: 999.weeks,
-          handleTouch: true,
-        );
-      } else {
-        SchedulerBinding.instance.addPostFrameCallback((_) {
-          connectivityToastFuture = showToastWidget(
-            noConnectivityWidget,
-            duration: 999.weeks,
-            handleTouch: true,
-          );
-        });
-      }
+    if (result == ConnectivityResult.none && connectivityToastFuture == null) {
+      connectivityToastFuture ??= showNoConnectivityDialog;
     } else {
-      if (Instances.connectivityResult == ConnectivityResult.none) {
-        connectivityToastFuture?.dismiss(showAnim: true);
+      connectivityToastFuture?.dismiss(showAnim: true);
+      if (connectivityToastFuture != null) {
+        connectivityToastFuture = null;
       }
     }
   }
 
+  ToastFuture get showNoConnectivityDialog => showToastWidget(
+        noConnectivityWidget,
+        duration: 999.weeks,
+        handleTouch: true,
+      );
+
   Widget get noConnectivityWidget => Material(
         color: Colors.black26,
         child: BackdropFilter(