setTimeout with function

Question

I'm trying to create a bookmarklet to open links with specific text, but I'm facing a problem with the setTimeout portion ...

javascript:(function(){
    function clickLink(link) {
        if (document.createEvent) {
            var event=document.createEvent("MouseEvents"); 
            event.initMouseEvent("click", true, true, window, 0, 0, 0, 0, 0, false, false, false, false, 0, null);
            link.dispatchEvent(event);
        } else if (link.fireEvent) {
            link.fireEvent("onclick");
        }
    } 

    l=document.links;
    for(var i=0;i<l.length;++i) {
        var lL=l[i].innerHTML.toLowerCase(); 
        if(lL.indexOf("click here")!=-1 || lL.indexOf("how")!=-1) 
             setTimeout(function() {clickLink(l[i]) }, 1000);
        }
    }; 
})();

If I try setTimeout(clickLink(l[i]), 1000);, then it works, but it runs immediately rather than waiting for the timeout!

Also want to know one more thing that clicking links this way will fire mousedown/mouseup event? If not then how can I programmatically click links which will also fire mousedown/mouseup events?

looks like you're putting all that js into an onclick attribute of a link or a button. It might be easier for you to debug if you put it into a function and call that function instead. — Simon
– Simon, Commented Sep 17, 2012 at 14:24

user1675187 · Accepted Answer · 2012-09-17 14:27:00Z

1

You're coming across the usual "loop variable in a lambda" problem: by the time the timeout is done, i will be past the end of document.links and l[i] will be undefined. You could just wrap that part in a(nother) function:

(function(item) {
    setTimeout(function() { clickLink(item); }, 1000);
})(l[i]);

answered Sep 17, 2012 at 14:27

user1675187

Sign up to request clarification or add additional context in comments.

3 Comments

James Smith Over a year ago

That solves the problem.. Thanks. BTW clicking links this way will fire mouseup/mousedown event?

user1675187 Over a year ago

@JamesSmith: No, it won't. You'll need separate events for those.

James Smith Over a year ago

Can you please tell me how can I do that? I also need to fire the mousedown/mouseup events while clicking links by bookmarklet?

Collectives™ on Stack Overflow

setTimeout with function

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related