kaihendry / greptweet

Sane twitter backup and search
https://greptweet.com/
Other
76 stars 10 forks source link

Linkify urls on grep page #38

Closed JamieKitson closed 10 years ago

JamieKitson commented 10 years ago
  1. Move the link to the tweet to the datetime text.
  2. Add links to urls in tweets. This can crudely be done with:
    $status_text = preg_replace(
     '/(https?:\/\/\S+)/',
     '',
     $status_text
    );
    
kaihendry commented 10 years ago

I always thought it was a hotbed of complexity to linkify URLs so I was always happy to defer that to twitter's UI.

I'm a bit bothered by the way the datetime text is rendered at the end of the line. Doesn't look quite right to me. Worried that if that becomes the link to the tweet, people might have more trouble that they should copying a link to the canonical URL of the tweet.

kaihendry commented 10 years ago

I don't want to linkify text.