-
Notifications
You must be signed in to change notification settings - Fork 9.8k
Update dart doc for forceSafariVC for the usage of universal links on iOS #1058
Conversation
@@ -20,9 +20,10 @@ const MethodChannel _channel = MethodChannel('plugins.flutter.io/url_launcher'); | |||
/// [forceSafariVC] is only used in iOS. If unset, the launcher opens web URLs | |||
/// in the safari VC, anything else is opened using the default handler on the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While you're here, consider replacing safari VC
with Safari view controller
.
@@ -20,9 +20,10 @@ const MethodChannel _channel = MethodChannel('plugins.flutter.io/url_launcher'); | |||
/// [forceSafariVC] is only used in iOS. If unset, the launcher opens web URLs |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To make it a tiny bit more obvious, maybe: By default (when unset), the launcher opens HTTP/HTTPS URLs
...
/// If false, the URL is opened in the default browser of the phone. Note that | ||
/// to work with universal links on iOS, this must be set to false to let | ||
/// the platform's system handle the URL. Set this to false if you want to | ||
/// use the cookies/context of the main browser of the app(such as SSO flows). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add a space between app
and (such as SSO flows)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM modulo a couple minor nitpicks.
… iOS (flutter#1058) Update dart document to explain forceSafariVC flag regarding universal link on IOS There are numbers of users having problem on launch universal links on iOS. It is most likely due to the unawareness of forceSafariVC is true by default and safariVC will not handle universal links.
…links on iOS (flutter#1058)" This reverts commit dcc6f24.
There are numbers of users having problem on launch universal links on iOS.
It is most likely due to the unawareness of forceSafariVC is true by default and safariVC will not handle universal links.
Adding document to explain this particular usage.