Javascript is automatically clicking in the EventSystem? What did I do wrong?

Question

The code is supposed to redirect the user to another internal page. However when the page loads with the code in it it automatically redirects the user without the user having to click the button.

I'm pretty new. I don't know what to try. I think it should work as is.

document.querySelector(".choice").addEventListener('apple', Redirect());

function Redirect()
{
    window.location = "challenge.html";
}

The expected result is that the user would click the text that contain the choice class and it would redirect the user to the challenge page.

The actual result is when the page loads it automatically redirects the user to the challenge page without the user having to click the button.

Where it says 'apple' i put 'click' i was just trying to see what would happen if i changed the click to something else. — Chris Klutts, Jan 08 '19 at 19:49
cont above.. overwise your executing the function immediatly instead of when clicked.. — Keith, Jan 08 '19 at 19:52

score 0 · Answer 1 · answered Jan 08 '19 at 19:51

You are invoking the event handler in this line:

document.querySelector(".choice").addEventListener('apple', Redirect());

Remove the parentheses:

document.querySelector(".choice").addEventListener('apple', Redirect);

You need to pass a reference to addEventListener otherwise with your current setup not only will the user be redirected immediately after the listener is registered, the button will also do nothing because the Redirect function is not returning something

THANK YOU ALL SO MUCH! I LOVE YOU – Chris Klutts Jan 08 '19 at 19:59 — Chris Klutts, Jan 08 '19 at 19:59

Javascript is automatically clicking in the EventSystem? What did I do wrong?

1 Answers1