feat(app): test if can pop for showing back button

hpi-dhc · Jan 3, 2025 · e7494c2 · e7494c2
1 parent 749985b
commit e7494c2
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 8 deletions.
diff --git a/app/lib/common/widgets/tutorial/show_app_tour.dart b/app/lib/common/widgets/tutorial/show_app_tour.dart
@@ -90,11 +90,11 @@ FutureOr<void> showAppTour(
       onClose: revisiting
         ? null
         : () async {
-          MetaData.instance.tutorialDone = true;
-          await MetaData.save();
-          // ignore: use_build_context_synchronously
-          await overwriteRoutes(context, nextPage: MainRoute());
-        },
+            MetaData.instance.tutorialDone = true;
+            await MetaData.save();
+            // ignore: use_build_context_synchronously
+            await overwriteRoutes(context, nextPage: MainRoute());
+          },
       lastNextButtonText: lastNextButtonText,
       firstBackButtonText: revisiting ? null : context.l10n.onboarding_prev,
     );
diff --git a/app/lib/common/widgets/tutorial/tutorial_builder.dart b/app/lib/common/widgets/tutorial/tutorial_builder.dart
@@ -145,8 +145,13 @@ class TutorialBuilder extends HookWidget {
     ValueNotifier<int> currentPageIndex,
   ) {
     final isFirstPage = currentPageIndex.value == 0;
-    final showFirstButton =
-      !isFirstPage || firstBackButtonText.isNotNullOrBlank;
+    final showFirstButton = !isFirstPage || (
+      firstBackButtonText.isNotNullOrBlank &&
+      context.router.canPop(
+        ignoreChildRoutes: true,
+        ignorePagelessRoutes: true,
+      )
+    );
     final isLastPage = currentPageIndex.value == pages.length - 1;
     final directionButtonTextStyle =
       PharMeTheme.textTheme.titleLarge!.copyWith(fontSize: 20);
@@ -164,7 +169,9 @@ class TutorialBuilder extends HookWidget {
               initiateRouteBack();
               final currentRoute = context.router.current.name;
               context.router.popUntil(
-                (route) => route.settings.name != null && route.settings.name != currentRoute,
+                (route) =>
+                  route.settings.name != null &&
+                  route.settings.name != currentRoute,
               );
             }
             : () => currentPageIndex.value = currentPageIndex.value - 1,