onmouseover function not working

Airshow 416 WiFi Lounge Lizard Team Colleague · 2011-08-14T15:37:09+00:00

MDanz,

I see you have marked this solved but here's a more efficient way to structure the code:

//First, frame everything inside an onload handler.
onload = function(){
	//It's much simpler to keep the slides in an array
	var slides = [];
	var n = 10;//number of slides
	var current = -1;
	var timeout;//as an "outer"	variable, timeout is accessible by every instance of nextslide.
	
	//Attach mouseover and mouseout handlers just once for each slide
	for(var i=0; i<n; i++){
		slides[i] = document.getElementById('slide' + i);//find each slide in the DOM just once
		if(slides[i]){//if the DOM element exists
			slides[i].onmouseover = function() { clearTimeout(timeout); };
			slides[i].onmouseout = nextslide;
		}
	}
	
	function nextslide() {
		if(slides[current]){//if the dom element exists
			slides[current].style.display = 'none';//hide current slide
		}
		current = (current+1) % slides.length;//Calculate next index, with wraparound to 0
		if(slides[current]){//if the dom element exists
			slides[current].style.display = 'block';//show next slide
		}
		timeout = setTimeout(nextslide, 2500);
	}
	nextslide();//Start the slideshow.
};

<body><!-- no need for onload="..." here -->

Airshow