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

Add call number action #1698

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Add call number action #1698

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 24, 2019

Description:

This patch adds the ability to call numbers just like sending e-mails. Instead of mailto it uses the tel uri scheme (see https://tools.ietf.org/html/rfc3966). On Windows 10 this prompts for a phone app which could be "MyPhone" app or a softphone like MicroSIP (tested). Support for tel-uris is whacky, but seems to get more common. The patch assumes phone numbers are entered in a RFC3966 compliant-ish with only minimal sanitation done in XSLTs.

Todo:

  • Check if this might be suitable for inclusion. Is a issues ticket required first?
  • Include submodule changes for UI text
  • Update call icon (uses copy of mail icon by now). Is there a default icon set to use?
  • Update XSLT to clean numbers.
  • Test

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.

None yet

0 participants