Skip to content

[Prototype] Change default link open behaviour to in-app browser by default#232

Draft
markmur wants to merge 2 commits into
mainfrom
change-link-open-behaviour
Draft

[Prototype] Change default link open behaviour to in-app browser by default#232
markmur wants to merge 2 commits into
mainfrom
change-link-open-behaviour

Conversation

@markmur
Copy link
Copy Markdown
Contributor

@markmur markmur commented May 28, 2026

Delegated web links (window.open and similar) previously bounced buyers out to the platform's external browser. That breaks flow for things like terms/policy pages opened mid-checkout. This PR flips the default so HTTPS links open in an in-app surface — SFSafariViewController on iOS, Custom Tabs on Android — while mailto:, tel:, and custom schemes continue routing through UIApplication.open / Intent.ACTION_VIEW. Merchants who want the old behavior can register a CheckoutProtocol.windowOpen handler.

Try the sample apps and tap a delegated link in checkout (e.g. a policy link). It should now stay in-app. The settings screen has been updated to let you toggle the alternative "External app" handler.

@markmur markmur self-assigned this May 28, 2026
@markmur markmur changed the title Change link open behaviour to open in-app by default [Protocol] Change default link open behaviour to in-app browser by default May 28, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 28, 2026

React Native — Coverage Report

Lines Statements Branches Functions
Coverage: 92%
91.59% (316/345) 87.25% (178/204) 100% (82/82)

@markmur markmur changed the title [Protocol] Change default link open behaviour to in-app browser by default [Prototype] Change default link open behaviour to in-app browser by default May 28, 2026
@markmur markmur force-pushed the change-link-open-behaviour branch from 59568ed to 9931f99 Compare May 28, 2026 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant