Skip to content
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

(optional) In app browser #2490

Merged
merged 4 commits into from
Jan 12, 2024
Merged

(optional) In app browser #2490

merged 4 commits into from
Jan 12, 2024

Conversation

pfrazee
Copy link
Collaborator

@pfrazee pfrazee commented Jan 11, 2024

Updated #2470

  • Tweak the UI a bit
  • Remove the .po file updates

CleanShot 2024-01-10 at 23 49 23@2x


Original PR

Adds expo-web-browser. Keeps current behaviour on web, and does not get used unless the user specifically opts in.

When opening a link, if there's no saved preference for the in app browser it will open a model asking if you want to open in the in app browser or in the default browser. A toggle has been added to the settings to change this setting later.

Simulator Screenshot - iPhone 15 - 2024-01-10 at 14 00 50

@pfrazee pfrazee force-pushed the mozzius-in-app-browser branch from 897a480 to 1ae22eb Compare January 11, 2024 07:53
@pfrazee pfrazee merged commit 998ee29 into main Jan 12, 2024
4 checks passed
@pfrazee pfrazee deleted the mozzius-in-app-browser branch January 12, 2024 18:19
estrattonbailey added a commit that referenced this pull request Jan 18, 2024
* origin/main: (44 commits)
  Patch `@lingui/core` to fix `unraw` import resolution error (#2548)
  1.65
  Fix the fallback to discover behavior on the home feed (#2546)
  Bump android version code
  Bump ios build number
  Add a new home feed-api wrapper and give a header indicating the fallback behavior (#2534)
  Add accept-language header (#2457)
  rss: filter out replies server-side (#2518)
  feat: show muted/blocked status on list card (#2523)
  1.64 (#2521)
  Bump [email protected] (#2519)
  Create a profile record on new user (#2520)
  fix: truncate long email address (#2493)
  fix: set html lang according to app language (#2496)
  Ensure sessions and tokens are synced between tabs (#2498)
  package.json: cp --verbose doesn't exist on macos (#2501)
  Suggest post language correction (#2486)
  (optional) In app browser  (#2490)
  Toggle minimal shell on any scroll for web (#2499)
  ✨ New report type, appeal (#2455)
  ...
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.

3 participants