Comments on: Detect external links & open in new window without “target blank” attribute

By: Jasper

Jasper — Thu, 23 Oct 2008 14:17:11 +0000

haha, works perfectly! Nice one!

By: Daze

Daze — Thu, 23 Oct 2008 07:48:20 +0000

Woordd! How’d you like those Apples!!

By: Daze

Daze — Thu, 23 Oct 2008 07:46:33 +0000

Also, just a thought. What happens if I put in a link to http://www.bbc.co.uk/radio1 in the comments area… I’ll add the same jQuery code as above I think…. ?

By: Daze

Daze — Thu, 23 Oct 2008 07:45:09 +0000

Ah indeed, I did notice something about do1.uk.com in there, but wasn’t quite sure what it was about. At the moment I’m just watching out when I create links in my posts to make sure the HREF attr begins with /page if its referencing a post or page on this site, or otherwise it’ll begin with http:// of course.

I guess what you’ve done there is a little more fool-proof, nicely done

By: Jasper

Jasper — Thu, 23 Oct 2008 07:41:07 +0000

it didn’t come across too well because of the comment formatting, but I also put checking in there for the current domain to check for a true external link:

if (this.get(â€™hrefâ€™).toString().match(/^http:\/\/[(www.)]*yourdomain\.com/) == null)