Update urchin code

EDIT: not Urchin, google Analytics


Example pulled from my source code:

Code:

Content Relevant URLs by <a onClick="javascript:urchinTracker ('/outgoing/http_www_crawlability_com_vbseo_');" rel="nofollow" href="http://www.crawlability.com/vbseo/">vBSEO</a>

Not only does this break XHTML 1.0 compatibility, it is also highly in-accessible.

Please change to:

Code:

Content Relevant URLs by <a onclick="urchinTracker('/outgoing/http_www_crawlability_com_vbseo_');" rel="nofollow" href="http://www.crawlability.com/vbseo/">vBSEO</a>

This will fix the XHTML, and will make the onclick event accesible for non-js people, and still prevent double windows/links in js enabled browsers.

Hello Brian,

the lowecase "onclick" was already implemented, I will remove "javascript:" as well.
Thanks!

and don't forget the return false; too This will prevent the double page load effect.

Edit: ignore :P

Hm.. but I think that link will not be opened when false is returned by onclick handler?

ah, you're right. The urchin call doesn't actually GO anywhere, so it still needs to hit the href, yes. Edited my post above.

Ok