cgrinds / dripdripdrop

A simple, fast, Tiny Tiny RSS reader

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Auto urlize links in articles.

hobeone opened this issue · comments

I subscribe to a few twitter feeds through their api:
http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=XXXXX

It's common for some people to have a short status + a link to a photo on facebook or flickr. They aren't full HTML links though just text of the url. It would be nice to automatically turn them into clickable links in the DDD ui.

Here's an example feed:
http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=ArgonautCycles

I looked at this and the tt-rss server does this currently.
However, the server only changes text links to urls when santizie=true. I could add the functionality client-side but not sure that makes sense. Any thoughts?

I have a working version in the linkify branch. Check it out and let me know what you think.

My only concern is performance since it will be applied indiscriminately to all article content but that's just a gut feeling I haven't profiled anything yet.

I have a working version in the linkify branch if you want to check it
out.

-Chris

On 05/26, hobeone wrote:

I subscribe to a few twitter feeds through their api:
http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=XXXXX

It's common for some people to have a short status + a link to a photo on facebook or flickr. They aren't full HTML links though just text of the url. It would be nice to automatically turn them into clickable links in the DDD ui.

Here's an example feed:
http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=ArgonautCycles


Reply to this email directly or view it on GitHub:
#15

Chris Grindstaff | http://gstaff.org

I think you need to checkin linkify.js too?

On Sun, May 26, 2013 at 4:16 PM, Chris Grindstaff
notifications@github.comwrote:

I have a working version in the linkify branch if you want to check it
out.

-Chris

On 05/26, hobeone wrote:

I subscribe to a few twitter feeds through their api:
http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=XXXXX

It's common for some people to have a short status + a link to a photo
on facebook or flickr. They aren't full HTML links though just text of the
url. It would be nice to automatically turn them into clickable links in
the DDD ui.

Here's an example feed:

http://api.twitter.com/1/statuses/user_timeline.rss?screen_name=ArgonautCycles


Reply to this email directly or view it on GitHub:
#15

Chris Grindstaff | http://gstaff.org


Reply to this email directly or view it on GitHubhttps://github.com//issues/15#issuecomment-18471370
.

Daniel Hobe hobe@gmail.com

Opps. There now

LGTM

Works on the few feeds I tested it on.

On Mon, May 27, 2013 at 9:35 AM, Chris Grindstaff
notifications@github.comwrote:

Opps. There now

?
Reply to this email directly or view it on GitHubhttps://github.com//issues/15#issuecomment-18506366
.

Daniel Hobe hobe@gmail.com