Fix lint warnings

proninyaroslav · Aug 10, 2024 · 1164b36 · 1164b36
1 parent 32b984c
commit 1164b36
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 14 deletions.
diff --git a/lib/ui/app_router.dart b/lib/ui/app_router.dart
@@ -212,17 +212,14 @@ class AppRouterDelegate extends RouterDelegate<AppRoutePath>
 
   @override
   Widget build(BuildContext context) {
+    final pages = _buildPages(context);
+
     return Navigator(
       key: navigatorKey,
       reportsRouteUpdateToEngine: true,
-      pages: _buildPages(context),
-      onPopPage: (route, result) {
-        if (!route.didPop(result)) {
-          return false;
-        }
+      pages: pages,
+      onDidRemovePage: (page) {
         _popState();
-
-        return true;
       },
     );
   }

diff --git a/lib/ui/home/navigation_page.dart b/lib/ui/home/navigation_page.dart
@@ -1409,7 +1409,7 @@ class _ContextualAppBar extends StatelessWidget {
   Widget build(BuildContext context) {
     return PopScope(
       canPop: false,
-      onPopInvoked: (didPop) async {
+      onPopInvokedWithResult: (didPop, result) async {
         onBackPressed?.call();
       },
       child: AppBar(

diff --git a/lib/ui/settings/settings_page.dart b/lib/ui/settings/settings_page.dart
@@ -319,13 +319,8 @@ class _SettingsRouterDelegate extends RouterDelegate<AppRoutePathSettings>
       key: navigatorKey,
       reportsRouteUpdateToEngine: true,
       pages: _buildPages(context),
-      onPopPage: (route, result) {
-        if (!route.didPop(result)) {
-          return false;
-        }
+      onDidRemovePage: (page) {
         _currentState = const _SettingsRouterState.home();
-
-        return true;
       },
     );
   }