+1

Please add it to Roundcube 1.0-RC/stable. It is really useful for mobile devices or VOIP calls on desktop computers. IMO all non-numeric characters except the leading + should be stripped off to receive urls like ...

<a href="tel:+31123456789">[Unformatted as it is in the addressbook]</a>

Also leading '00' should be replaced by '+' in the formatting function.

Hello, i have modified the show.inc file for append the support for
tel://url in addressbook.
If you want add the modification to your code, it's ok for me.

Cheers,

--- old/program/steps/addressbook/show.inc
+++ new/program/steps/addressbook/show.inc

@@ -137,7 +137,7 @@
              'name'    => rcube_label('properties'),
              'content' => array(
                'email' => array('size' => $i_size, 'render_func' =>
'rcmail_render_email_value'),
-              'phone' => array('size' => $i_size),
+              'phone' => array('size' => $i_size, 'render_func' =>
'rcmail_render_tel_value'),
                'address' => array(),
                'website' => array('size' => $i_size, 'render_func' =>
'rcmail_render_url_value'),
                'im' => array('size' => $i_size),
@@ -187,6 +187,14 @@
      ), Q($email));
  }

+function rcmail_render_tel_value($tel, $col)
+{
+    return html::a(array(
+        'href' => 'tel:' . $tel,
+        'class' => 'tel',
+    ), Q($tel));
+}
+

  function rcmail_render_url_value($url, $col)
  {

_______________________________________________
Roundcube Development discussion mailing list
[email protected]
http://lists.roundcube.net/mailman/listinfo/dev

Reply via email to