heiseonline / shariff

👮 Shariff enables website users to share their favorite content without compromising their privacy.
http://ct.de/-2467514
MIT License
1.4k stars 177 forks source link

Don't show '<' and '>' around a linked shared by email #98

Closed Twiek closed 9 years ago

Twiek commented 9 years ago

You can see it on your demo page:

bildschirmfoto 2015-07-17 um 13 47 28

If I click the email button I get

bildschirmfoto 2015-07-17 um 13 47 47

Instead of '<http://www.heise.de/>' I would like to see just 'http://www.heise.de/'

craiq commented 9 years ago

it's added on purpose. So there might be a reason for this.

You can check with my build, if there are error's: https://github.com/craiq/shariff/

but I need to warn you, my build uses completely new css the demo might be enough to test http://craiq.github.io/shariff/

sehkunde commented 9 years ago

Hi,

some mail clients line wrap long urls and break them. Wrapping them in angle brackets can help. See this recommendation: http://www.w3.org/Addressing/URL/5.1_Wrappers.html

You should be able to suppress the brackets by passing a modified function for the property "mailBody". This is the default function:

mailBody: function() { return '<' + this.getURL() + '>'; },

I hope that answers your question.