Replace the onUrlNavigation callback with internal logic

Author: Franco Bugnano

lib/src/chatbox.dart

@@ -9,6 +9,7 @@ import 'package:flutter/gestures.dart';
 
 import 'package:flutter_inappwebview/flutter_inappwebview.dart';
 import 'package:permission_handler/permission_handler.dart';
+import 'package:url_launcher/url_launcher.dart';
 
 import './session.dart';
 import './conversation.dart';
@@ -88,7 +89,6 @@ class ChatBox extends StatefulWidget {
   final TranslationToggledHandler? onTranslationToggled;
   final LoadingStateHandler? onLoadingStateChanged;
   final Map<String, MessageActionHandler>? onCustomMessageAction;
-  final NavigationHandler? onUrlNavigation;
 
   const ChatBox({
     Key? key,
@@ -107,7 +107,6 @@ class ChatBox extends StatefulWidget {
     this.onTranslationToggled,
     this.onLoadingStateChanged,
     this.onCustomMessageAction,
-    this.onUrlNavigation,
   }) : super(key: key);
 
   @override
@@ -235,7 +234,18 @@ class ChatBoxState extends State<ChatBox> {
         // We need only the VerticalDragGestureRecognizer in order to be able to scroll through the messages
         Factory(() => VerticalDragGestureRecognizer()),
       },
-      shouldOverrideUrlLoading: _shouldNavigateToUrl,
+      shouldOverrideUrlLoading: (InAppWebViewController controller, NavigationAction navigationAction) async {
+        if (navigationAction.navigationType == NavigationType.LINK_ACTIVATED) {
+          if (await launchUrl(navigationAction.request.url!)) {
+            // We launched the browser, so we don't navigate to the URL in the WebView
+            return NavigationActionPolicy.CANCEL;
+          } else {
+            // We couldn't launch the external browser, so as a fallback we're using the default action
+            return NavigationActionPolicy.ALLOW;
+          }
+        }
+        return NavigationActionPolicy.ALLOW;
+      },
     );
   }
 
@@ -466,26 +476,6 @@ class ChatBoxState extends State<ChatBox> {
     widget.onCustomMessageAction?[action]?.call(MessageActionEvent.fromJson(jsonMessage));
   }
 
-  Future<NavigationActionPolicy?> _shouldNavigateToUrl(
-    InAppWebViewController controller,
-    NavigationAction navigationAction,
-  ) async {
-    if (widget.onUrlNavigation != null) {
-      final UrlNavigationAction action = widget.onUrlNavigation!(
-        UrlNavigationRequest(navigationAction.request.url!.rawValue),
-      );
-
-      switch (action) {
-        case UrlNavigationAction.allow:
-          return NavigationActionPolicy.ALLOW;
-        case UrlNavigationAction.deny:
-          return NavigationActionPolicy.CANCEL;
-      }
-    }
-
-    return NavigationActionPolicy.ALLOW;
-  }
-
   /// For internal use only. Implementation detail that may change anytime.
   ///
   /// Return a string with a unique ID

pubspec.lock

@@ -166,7 +166,7 @@ packages:
       name: flutter_inappwebview
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "6.0.0-beta.12"
+    version: "6.0.0-beta.22"
   flutter_inappwebview_internal_annotations:
     dependency: transitive
     description:
@@ -180,21 +180,21 @@ packages:
       name: flutter_local_notifications
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "9.9.1"
+    version: "13.0.0"
   flutter_local_notifications_linux:
     dependency: transitive
     description:
       name: flutter_local_notifications_linux
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "0.5.1"
+    version: "3.0.0"
   flutter_local_notifications_platform_interface:
     dependency: transitive
     description:
       name: flutter_local_notifications_platform_interface
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "5.0.0"
+    version: "6.0.0"
   flutter_test:
     dependency: "direct dev"
     description: flutter
@@ -370,14 +370,70 @@ packages:
       name: timezone
       url: "https://pub.dartlang.org"
     source: hosted
-    version: "0.8.0"
+    version: "0.9.1"
   typed_data:
     dependency: transitive
     description:
       name: typed_data
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.3.0"
+  url_launcher:
+    dependency: "direct main"
+    description:
+      name: url_launcher
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.1.8"
+  url_launcher_android:
+    dependency: transitive
+    description:
+      name: url_launcher_android
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.0.23"
+  url_launcher_ios:
+    dependency: transitive
+    description:
+      name: url_launcher_ios
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "6.0.18"
+  url_launcher_linux:
+    dependency: transitive
+    description:
+      name: url_launcher_linux
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "3.0.2"
+  url_launcher_macos:
+    dependency: transitive
+    description:
+      name: url_launcher_macos
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "3.0.2"
+  url_launcher_platform_interface:
+    dependency: transitive
+    description:
+      name: url_launcher_platform_interface
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.1.1"
+  url_launcher_web:
+    dependency: transitive
+    description:
+      name: url_launcher_web
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "2.0.14"
+  url_launcher_windows:
+    dependency: transitive
+    description:
+      name: url_launcher_windows
+      url: "https://pub.dartlang.org"
+    source: hosted
+    version: "3.0.2"
   vector_math:
     dependency: transitive
     description:

pubspec.yaml

@@ -15,14 +15,15 @@ dependencies:
   flutter:
     sdk: flutter
 
-  flutter_inappwebview: ^6.0.0-beta.12
+  flutter_inappwebview: ^6.0.0-beta.22
   crypto: ^3.0.1
   firebase_core: ^2.3.0
   firebase_messaging: ^14.1.1
-  flutter_local_notifications: ^9.6.0
+  flutter_local_notifications: ^13.0.0
   http: ^0.13.4
   flutter_apns_only: ^1.5.2
   permission_handler: ^10.1.0
+  url_launcher: ^6.1.8
 
 dev_dependencies:
   flutter_test: