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

Mailto protocol handler vanished #2535

Closed
ChristophWurst opened this issue Jan 23, 2020 · 1 comment · Fixed by #2539
Closed

Mailto protocol handler vanished #2535

ChristophWurst opened this issue Jan 23, 2020 · 1 comment · Fixed by #2539

Comments

@ChristophWurst
Copy link
Member

Expected behavior

It should be possible for the app to register as mailto: protocol handler. This worked up until we migrated to Vue.

Actual behavior

There is no handler.

Solution

The app should [register a protocol handler]https://developer.mozilla.org/en-US/docs/Web/API/Navigator/registerProtocolHandler). We used to do this as soon as the app starts https://github.com/nextcloud/mail/blob/v0.11.0/js/app.js#L64-L73. But I was never a fan of this as it's making the first run experience worse with an additional dialogue.

Thus we should just add a setting to the bottom left. Like a button "Register as application for mail links". When the user clicks this, we run https://github.com/nextcloud/mail/blob/v0.11.0/js/app.js#L64-L73 (maybe without the theme thing because that API is ugly 🙈).

Mail app

Mail app version:

Anything after 0.11

@ChristophWurst
Copy link
Member Author

@GretaD for testing you can use the example links at http://mc-computing.com/HTML_Examples/MailTo.htm. Once the protocol handler has been registered Mail should open when you click one of the links :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants