This is a really small patch. It adds one line of Google Analytics code to track clicks on the pivots block (for better formatting, it's actually 3 lines of code)

Another tiny change: reduce the number of links in the pivots block from 10 down to 5.

Please apply this small patch to CVS. Thank you!

CommentFileSizeAuthor
pivots_block.module.patch1.3 KBdanithaca

Comments

danithaca’s picture

danithaca commented

SInce d.o. is now using the old Google Analytics toolset, we are also using the old GA toolset for consistency. The GA code in the patch followed this article.
http://www.google.com/support/googleanalytics/bin/answer.py?answer=74985

Also, GA documentation has this note:

"Important: if your pages include a call to urchinTracker(), utmLinker(), utmSetTrans(), or utmLinkPost(), the tracking code must be placed in your page's code above any of these calls. In these cases the tracking code can be placed anywhere between the opening tag and the JavaScript call. "
http://www.google.com/support/googleanalytics/bin/answer.py?answer=74954

I'm not sure if it has any implication to the current GA deployment on d.o.

kbahey’s picture

kbahey commented

I think the second answer does not apply because this is an onClick event which will happen only after GA's js file loads.

So, I committed it.

Can someone get hold of killes to apply it to d.o?

gerhard killesreiter’s picture

gerhard killesreiter commented
Status: Active » Fixed
Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.

  • Commit d6fba06 on 5.x-1.x, 6.x-1.x, 6.x-3.x, 7.x-3.x-dev by kbahey: 
    #322228 Add Google Analytics tracking to clicks on pivots.