fix: truely handle popping events

Author: realth000

lib/features/home/view/home_page.dart

@@ -26,6 +26,7 @@ import 'package:tsdm_client/utils/logger.dart';
 import 'package:tsdm_client/utils/platform.dart';
 import 'package:tsdm_client/utils/show_dialog.dart';
 import 'package:tsdm_client/utils/show_toast.dart';
+import 'package:tsdm_client/widgets/safe_pop_scope.dart';
 import 'package:tsdm_client/widgets/shutdown.dart';
 
 const _drawerWidth = 250.0;
@@ -278,7 +279,7 @@ class _HomePageState extends State<HomePage> with LoggerMixin {
                 // Unreachable
                 return;
               },
-              child: child,
+              child: SafePopScope(path: '<Home Page>', child: child),
             );
           },
         ),

lib/features/root/bloc/root_location_cubit.dart

@@ -76,10 +76,4 @@ final class RootLocationCubit extends Cubit<RootLocationState> with LoggerMixin
     await _sub.cancel();
     return super.close();
   }
-
-  @override
-  void onChange(Change<RootLocationState> change) {
-    super.onChange(change);
-    warning('>>> $change');
-  }
 }

lib/features/root/view/root_page.dart

@@ -2,7 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:tsdm_client/features/root/models/models.dart';
 import 'package:tsdm_client/features/root/stream/root_location_stream.dart';
 import 'package:tsdm_client/utils/logger.dart';
-import 'package:tsdm_client/widgets/shutdown.dart';
+import 'package:tsdm_client/widgets/safe_pop_scope.dart';
 
 /// A top-level wrapper page for showing messages or provide functionalities to
 /// all pages across the app.
@@ -29,18 +29,6 @@ class _RootPageState extends State<RootPage> with LoggerMixin {
 
   @override
   Widget build(BuildContext context) {
-    return BackButtonListener(
-      onBackButtonPressed: () async {
-        // App wide popping events interceptor, handles all popping events and notify the listener above.
-        rootLocationStream.add(const RootLocationEventLeavingLast());
-        if (!context.mounted) {
-          // Well, leave it here.
-          await exitApp();
-          return true;
-        }
-        return true;
-      },
-      child: widget.child,
-    );
+    return SafePopScope(path: widget.path, child: widget.child);
   }
 }

lib/widgets/safe_pop_scope.dart

@@ -0,0 +1,29 @@
+import 'package:flutter/material.dart';
+import 'package:tsdm_client/features/root/models/models.dart';
+import 'package:tsdm_client/features/root/stream/root_location_stream.dart';
+
+/// Wrapper widget handle page popping events.
+class SafePopScope extends StatelessWidget {
+  /// Constructor.
+  const SafePopScope({required this.path, required this.child, super.key});
+
+  /// Route path of [child] page.
+  final String path;
+
+  /// Wrapped child widget.
+  final Widget child;
+
+  @override
+  Widget build(BuildContext context) {
+    return PopScope(
+      canPop: false,
+      onPopInvokedWithResult: (didPop, b) {
+        if (!didPop) {
+          // When popping event passed location check, `didPop` is true, do not recursively handle it.
+          rootLocationStream.add(const RootLocationEventLeavingLast());
+        }
+      },
+      child: child,
+    );
+  }
+}