Skip to content
This repository was archived by the owner on Mar 7, 2022. It is now read-only.

Commit 6cf2070

Browse files
committed
new facebook mobile_iframe option
1 parent 6a5f13a commit 6cf2070

5 files changed

+22
-8
lines changed

bower.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "angular-socialshare",
3-
"version": "2.3.6",
3+
"version": "2.3.7",
44
"description": "A social media url and content share module for angularjs.",
55
"authors": [
66
"Filippo Oretti <[email protected]",

dist/angular-socialshare.js

+7-6
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
/*
22
* angular-socialshare
3-
* 2.3.6
3+
* 2.3.7
44
*
55
* A social media url and content share module for angularjs.
66
* http://720kb.github.io/angular-socialshare
77
*
88
* MIT license
9-
* Fri Feb 17 2017
9+
* Sat Feb 18 2017
1010
*/
1111
/*global angular*/
1212
/*eslint no-loop-func:0, func-names:0*/
@@ -551,7 +551,7 @@
551551

552552
urlString += '&bcc=' + encodeURIComponent(attrs.socialshareBcc);
553553
}
554-
if($window.self !== $window.top) {
554+
if ($window.self !== $window.top) {
555555
$window.open(urlString, '_blank');
556556
} else {
557557
$window.open(urlString, '_self');
@@ -841,18 +841,19 @@
841841
}
842842
, manageSmsShare = function smsShare($window, attrs, element) {
843843

844-
if(attrs.socialshareText.indexOf('%') >= 0) {
844+
if (attrs.socialshareText.indexOf('%') >= 0) {
845845
$log.warn('sending sms text with "%" sign is not supported');
846846
}
847847

848848
var body = encodeURIComponent(attrs.socialshareText.replace('%',''))
849-
, toPhoneNumber = attrs.socialshareTo || '';
849+
, toPhoneNumber = attrs.socialshareTo || ''
850+
, urlString;
850851

851852
if (attrs.socialshareUrl) {
852853
body += encodeURIComponent(attrs.socialshareUrl);
853854
}
854855

855-
var urlString = 'sms:' + toPhoneNumber + '?&body=' + body;
856+
urlString = 'sms:' + toPhoneNumber + '?&body=' + body;
856857

857858
element.attr('href', urlString);
858859
element.attr('target', '_blank');

0 commit comments

Comments
 (0)