views:

121

answers:

3

Hi, I have the following code but the clear timeout doesn't work and I can't understand why, does anyone have any ideas? (Using the Prototype framework)

Thanks

function foo() {
    $("navigation").observe('mouseover',
     function (event) {
      clearTimeout(bar);
     }
    ).observe('mouseout',
     function (event) {
      setTimeout(bar, 1000);
     }
    );
}

function bar() {
    alert("hi");
}
+3  A: 

Because the clearTimeout function take the argument returned by the setTimeout function:

var t = null;
function foo() {
    $("navigation").observe('mouseover',
        function (event) {
            if (t != null) clearTimeout(t);
        }
    ).observe('mouseout',
        function (event) {
            t = setTimeout(bar, 1000);
        }
    );
}

function bar() {
    alert("hi");
}
Darin Dimitrov
A: 

You need to store the result of setTimeout in a variable, and use clearTimeout to clear that variable, not the function:

var timer;

function foo() {
    $("navigation").observe('mouseover',
        function (event) {
                clearTimeout(timer);
        }
    ).observe('mouseout',
        function (event) {
                timer = setTimeout(bar, 1000);
        }
    );
}

function bar() {
    alert("hi");
}
Doug Neiner
Ah I understand now, thanks guys :)
Rich
Hey, no problem. Be sure to mark the answer solved by choosing one of the answers as the "accepted" solution.
Doug Neiner
+2  A: 

See the mozilla docs on window.setTimeout():

setTimeout actually returns a reference which you can use to clear the timeout:

tId = setTimeout(bar, 1000);
clearTimeout(tId);
David Caunt