Pass urls stored in array to window.open in for loop - Javascript

Question

I'm trying to step through an array and pass my urls stored there to the window.open function. This is what I have.

$('a.linked').click(function(e) {
    e.preventDefault();
    var i;
    var urls = ['http://example.com', 'http://example2.com', 'http://example3.com'];

    for (i = 0; i < urls.length; i++) {
      var ts = i*3500;
      setTimeout(function(){window.open(urls[i]); }, ts);
  }
});

This is opening the new windows at the correct timing but I'm getting about:blank. I thought it might be not escaping quotes.

The duplicate answer says I'm losing my value of i However, when I remove the setTimeout I get all the new windows with the correct urls. The following code demonstrates not losing the value of i.

$('a.linkedinProfiles').click(function(e) {
    e.preventDefault();
    var i;
    var urls = ['http://example.com', 'http://example2.com', 'http://example3.com'];
    for (i = 0; i < urls.length; i++) {
    var ts = i*3500;
    {window.open(urls[i]); };   
    } });

I know this has to be something simple I am missing. Thanks in advance.

a few things are wrong here: don't defer opening the windows - they have to be open with a click in the stack. also you are losing the value for `i`. — Daniel A. White, Feb 08 '16 at 00:32
you need to use fully qualified URLs - including protocol (e.g. `http://example.com`). — Sam Axe, Feb 08 '16 at 00:32

score 0 · Answer 1 · answered Feb 08 '16 at 00:32

0

May not be the full solution, but you could try adding "http://" to the beginning of your URLs. Currently they will be opened as relative links.

answered Feb 08 '16 at 00:32

TW80000

1,507
1
11
18

sorry the urls in the example were shortened and I forgot to add http:// – Ryan Litwiller Feb 08 '16 at 01:22

Pass urls stored in array to window.open in for loop - Javascript

1 Answers1