-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Try to control which external browser/app is used on opening a link #74
Comments
On Android, the experience I'm accustomed to as a user is that once I've decided to open something in another app, the app I'm coming from doesn't attempt to specify which particular app I use — instead I get a piece of system UI that asks me, and I choose there. (Including an "always use this app" option, but that's still between me and the system.) It looks like this: So I think we probably don't want to try to usurp that Android behavior. On iOS I don't know what the right thing is, though. |
Hey, it could be that iOS acts similar and I just don't know about it. Lets see how it works just to be aware. |
Hmm, well — I'm no longer seeing that behavior! At least not when I open an On that Android docs page, their own example shows a choice between two browsers: I first noticed that I wasn't getting this behavior when I implemented #71, using an external browser ( Specifically, what I get is that the link leads directly to opening in the Chrome app. Which indeed I appear to have set as my default browser on the system. Anyway, for this issue's purposes the bottom line is the same: on Android there's already a systemwide way to choose the browser, which is under the user's control, so I don't think we'll want to try to usurp that. |
The second TODO, about the behavior of externalApplication, turns out to be just how things work these days on Android: #74 (comment) The first one, about the behavior of inAppWebView or platformDefault, I've filed as #279. I also added a number of details on the upstream issue, and cross-linked it with a couple of other issues that would be fixed by the same solution.
This will be a followup to:
@terpimost writes:
and adds:
The text was updated successfully, but these errors were encountered: