Code not working onmouseover

[wbevan20]

Hi

I have the following code for a dropdown menu that I am creating and the problem is that whilst the variables are being set correctly the for and if statement aren't functioning correctly.

Instead of allowing the dropdown menu to be activated onmouseover it flashes on/off briefly when the page is loaded and that's it.

Here is the code that I am working with:

[CODE=javascript]function gatherItems(c){
//alert("Function has been called and the id is: "+c);
container = document.getEle mentById(c);
items = container.getEl ementsByTagName ('a');
subs = container.getEl ementsByTagName ('div');
alert(items.len gth+" "+subs.leng th);
//alert(items.id) ;
for(var i=0;i<items.len gth;i++){
if(items[i].id){
items[i].onmouseover = show(items[i].id);
items[i].onmouseout = delay();
}
}
/*for(var s=0;s<subs.leng th;s++){
if(subs[s].id){
subs[s].onmouseover = canceldelay();
subs[s].onmouseout = delay();
}
}*/
}[/CODE]

Note: The functions being called onmouseover and onmouseout are working fine without this piece of code.

[gits]

please try the following explicit closure construct. i think it should work.

for(var i=0; i< items.length; i++) {
    if(items[i].id){
        items[i].onmouseover = function(idx) {
            return function() { show(items[idx].id) };
        }(i);

        items[i].onmouseout = delay();
    }
}

assigning handlers in a loop leads to always have i = items.length in the end. closuring the index explicitly should fix the issue.

[wbevan20]

That has solved the problem of the dropdown not appearing, however it now won't go away without refreshing the page

[gits]

?? what does the delay-method do? i suppose it should trigger a close action? while assigning the handler you mustn't execute that, assign it like this:

items[i].onmouseout = delay;

that way you assign a reference to the delay-function which then is executed on mouseout.

[JackieBolinsky]

Hi...
I hope once you wrap the script in $(document).rea dy(function () { }); this will work correctly. Thanks for the code,i run the code but there is a delay only when i mouseover menu3 and i need in others too,only when i mouseover and mouseout in specific divs the settimeout shall be disabled. If you have control over the code, it is a lot simpler to just build the delay into the event handler from the beginning than wrapping it in on the client side.

thanks,
jackie

[Removed URL]

[gits]

@Jackie:

huh - can you please explain what that means? how is that post related to the above posts? what is menu3, what are the specific divs and how could one have no control over his/her code etc.?

regards

[wbevan20]

Thanks for all your help, it now works perfectly :)