bug: 1.0.0-rc5 break tel: link

[henryavila]

Type: bug

Platform: android 4.4 webview

When I update ionic to RC5 my tel: simply stop to work on mobile (tested on Android).
I checked my config.xml, its ok. roollbak the ionic to RC4 and it back to work. Updated again to RC5 and it stop to work again. For some reason this update just broke the tel: links.

My code is this

<a class="button button-assertive" ng-if="item.phone" ng-href="tel:{{item.phone}}">
<i class="icon ion-ios-telephone-outline light"></i>
</a>

I'm using it for about four months without any problem. The problem came just when I updated the lib

[jeanmatthieud]

I saw it too.

[joshbuchea]

Incase it's helpful for anyone, in an Ionic project with RC5 that I'm currently working on I use a function to call the tel link and that still works.

document.location.href = 'tel:' + phone;

[jeanmatthieud]

It would be great to add "geo:" links and all supported links with the new "cordova-plugin-whitelist"

[olach]

Thanks for this fix. But do you really need to also add "tel:" to imgSrcSanitizationWhitelist? "Tel:" is only used on links, not images. :)

[henryavila]

Angular make images linkable too. It can be uses on images

Em qua, 13 de mai de 2015 05:42, Ola Christensson [email protected]
escreveu:

Thanks for this fix. But do you really need to also add "tel:" to
imgSrcSanitizationWhitelist? "Tel:" is only used on links, not images. :)

—
Reply to this email directly or view it on GitHub
#3647 (comment).