Skip to content

Fixed open article in browser call#807

Merged
David-Development merged 1 commit intonextcloud:masterfrom
emasty:fix_open_browser_behaviour
Jan 5, 2020
Merged

Fixed open article in browser call#807
David-Development merged 1 commit intonextcloud:masterfrom
emasty:fix_open_browser_behaviour

Conversation

@emasty
Copy link
Copy Markdown
Contributor

@emasty emasty commented Jan 2, 2020

Opening an article in browser was handled differently in different views.

  • When a link was pressed in the webview showing the article, the users browser settings were considered and the correct browser opened
  • When a link was opened using the menu entry, the article was always loaded in Chrome custom tabs, independend of user settings.

Both ways now use the same function and behaviour.

Opening an article in browser was handled differently in different views.
- When a link was pressed in the webview showing the article, the users browser settings were considered and the correct browser opened
- When a link was opened using the menu entry, the article was always loaded in Chrome custom tabs, independend of user settings.

Both ways now use the same function and behaviour.

Signed-off-by: emasty <emasty@gmail.com>
@emasty emasty mentioned this pull request Jan 5, 2020
@David-Development
Copy link
Copy Markdown
Member

Awesome, thank you! I noticed issues with it too but didn't have the time to look into it yet. Thank you for working on it! :)

@David-Development David-Development merged commit d8b7f51 into nextcloud:master Jan 5, 2020
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