My event handler is executed before the event is fired, why? [duplicate]

Question

I have a pop up window with a button. When I click the button I want something to happen, say an alert.

The issue I am having is that the onclick event fires as soon as the popup is launched and not when the button is clicked.

Here is the code. Any thoughts will be much appreciated.

var popup = open("", "Popup", "width=300,height=200");
var btn = popup.document.createElement("button");
btn.style.height = "50px";
btn.style.width = "150px";
popup.document.body.appendChild(btn);

btn.innerHTML="button1";
btn.onclick = alert("hello");

Grams

Answer 1

In your code

btn.onclick = alert("hello");

onclick is not fired. Just alert is executed immediately. You should wrap it into a function:

btn.onclick = function(){ alert("hello");}

Ahab

May I ask why is the first function being executed ? Looks like we are just assigning a function to the onclick event. – Haro

Answer 2

assign function to the btn.onclick event.

Obnoxious