In order for the click-tracking to work, the HTML in the theme must emit something like <div id="node-2" class="node"> (but not all themes do this). It might be worthwhile warning users about this, so they don't spend time wondering why the nodes that contain the links are not tracked. Perhaps something like:

NOTE: in order for the pop_links javascript to accurately track which nodes contain the external hyperlinks, your page HTML must contain <div> elements like this: <div id="node-1234" class="node"> (containing both the class=node and the id=node-NNN). Garland does this by default, but you can include this in your own theme by adding a link like this to your node.tpl.php:

<div id="node-<?php print $node->nid; ?>" class="node"> ?>

along with a

at the bottom. See Garland's node.tpl.php for example.

Hope this helps!

Comments

lyricnz’s picture

Please remove the trailing ?> from the code sample above, and change the second-last sentence bit to read: "along with a </div> at the bottom"

sikjoy’s picture

Hi lyricnz:

Very good suggestion. I'll add this to the README.txt and INSTALL.txt.

Thanks,

sicjoy

sikjoy’s picture

Version: 6.x-1.0 » 6.x-1.x-dev
Status: Needs review » Fixed

Added INSTALL.txt to module files and included this warning in the items list.

Anonymous’s picture

Status: Fixed » Closed (fixed)

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