Here's the page in question,

I'm trying to get rid of all the inline javascript on these pages,
So i'm making an array of id's of the images and trying to setup listeners for the popups.

Here's my script,

Code JavaScript:
var images = new Array (
	var len = images.length;
	for (var i=0; i<len; i++) {
		var element = document.getElementById(images[i]+'_thumb');
		addEvent(element, 'mouseover', show(images[i]), true);
		addEvent(element, 'mouseout', hide(images[i]), true);

In the page, for the top three images I have removed the inline eventListeners for testing the new method.

If people are unfamiliar with the addEvent function by Scott Andrews here it is,

Code JavaScript:
function addEvent(elm, evType, fn, useCapture)
// cross-browser event handling for IE5+, NS6+ and Mozilla 
// By Scott Andrew 
  if (elm.addEventListener) { 
    elm.addEventListener(evType, fn, useCapture); 
    return true; 
  } else if (elm.attachEvent) { 
    var r = elm.attachEvent('on' + evType, fn); 
    return r; 
  } else {
    elm['on' + evType] = fn;
The pristine markup nazi's are on my case

Any help?