Skip to content
This repository was archived by the owner on Feb 22, 2023. It is now read-only.

[url_launcher] forceSafariVC should be nullable to avoid breaking change #3354

Merged
mehmetf merged 2 commits into from
Dec 21, 2020
Merged

[url_launcher] forceSafariVC should be nullable to avoid breaking change #3354

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
db704fe
Fix breaking change
mehmetf Dec 20, 2020
c06a1e1
Version bump
mehmetf Dec 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Fix breaking change
  • Loading branch information
@mehmetf
mehmetf committed Dec 20, 2020
commit db704fef154a12b371cd7c0bd83c6344f6c940b6
4 changes: 2 additions & 2 deletions packages/url_launcher/url_launcher/lib/url_launcher.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ import 'package:url_launcher_platform_interface/url_launcher_platform_interface.
/// is set to true and the universal link failed to launch.
Future<bool> launch(
String urlString, {
bool forceSafariVC = true,
bool? forceSafariVC,
bool forceWebView = false,
bool enableJavaScript = false,
bool enableDomStorage = false,
Expand Down Expand Up @@ -95,7 +95,7 @@ Future<bool> launch(

final bool result = await UrlLauncherPlatform.instance.launch(
urlString,
useSafariVC: forceSafariVC,
useSafariVC: forceSafariVC ?? isWebURL,
useWebView: forceWebView,
enableJavaScript: enableJavaScript,
enableDomStorage: enableDomStorage,
Expand Down