Skip to content

Open links in different ways (browser and webview)#4343

Closed
shaurya2612 wants to merge 4 commits into
zulip:masterfrom
shaurya2612:4323
Closed

Open links in different ways (browser and webview)#4343
shaurya2612 wants to merge 4 commits into
zulip:masterfrom
shaurya2612:4323

Conversation

@shaurya2612
Copy link
Copy Markdown
Contributor

This PR is in continuation of #4336 and resolves issue #4323

I've added a seperate function called openLinkEmbedded for opening links in webview, as stated in -#4336 (comment) - I have made changes to open a webview in AuthScreen.js and webAuth.js.

There are two functions for opening links, openLinkEmbedded will open links in embedded webview for both android and ios while openLink will open links in browser for ios and in webview for android
As the previous commit (b19a686) introducedopenLinkEmbedded, webAuth and AuthScreen shall use it for a better user experience while logging in
@shaurya2612
Copy link
Copy Markdown
Contributor Author

shaurya2612 commented Dec 16, 2020

I went through all the openLink calls in the codebase and made the following changes accordingly :

  • WebLink.js (openLink -> openLinkEmbedded) --- I feel the forget password button gives a better overall experience when opened in a webview.
  • LightboxActionSheet.js & ShareImage.js (openLink -> openLinkEmbedded) --- changing this really didn't make much of a difference so i chose to stick with what was previously done.
  • messagesActions.js (openLink) --- opening these links in the browser seems like the best option to me
  • LegalScreen.js (openLink -> openLinkEmbedded)

@gnprice what do you think?

@chrisbobbe
Copy link
Copy Markdown
Collaborator

Closing, as this was taken care of in #4679.

@chrisbobbe chrisbobbe closed this Sep 7, 2021
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.

2 participants