add onclick event to newly added element in javascript

Question

Hi I have been trying to add onclick event to new elements I added with javascript. the problem is when I check document.body.innerHTML I can actually see the onclick=alert('blah') is added to the new element.

but when I click that element I don't see the alert box is working. in fact anything related to javascript is not working.. here is what I use to add new element:

function add_img()
{ 
var elemm = document.createElement('rvml:image'); 
elemm.src = 'blah.png';
elemm.className = 'rvml';
elemm.onclick = "alert('blah')";
document.body.appendChild(elemm);
elemm.id = "gogo";
elemm.style.position='absolute';
elemm.style.width=55;
elemm.style.height=55;
elemm.style.top=200;
elemm.style.left=300;
elemm.style.rotation=200; 
}

here is how I call this function:

<button onclick=add_img()>add image</button>

now the image draws perfectly inside the browser. but when I click the image I don't get that alert.

any help would be appreciated..

Answer 1

.onclick should be set to a function instead of a string. Try

elemm.onclick = function() { alert('blah'); };

instead.

Answer 2

Not sure but try :

elemm.addEventListener('click', function(){ alert('blah');}, false);

Answer 3

you can't assign an event by string. Use that:

elemm.onclick = function(){ alert('blah'); };

Answer 4

Short answer: you want to set the handler to a function:

elemm.onclick = function() { alert('blah'); };

Slightly longer answer: you'll have to write a few more lines of code to get that to work consistently across browsers.

The fact is that even the sligthly-longer-code that might solve that particular problem across a set of common browsers will still come with problems of its own. So if you don't care about cross-browser support, go with the totally short one. If you care about it and absolutely only want to get this one single thing working, go with a combination of addEventListener and attachEvent. If you want to be able to extensively create objects and add and remove event listeners throughout your code, and want that to work across browsers, you definitely want to delegate that responsibility to a library such as jQuery.

Answer 5

I don't think you can do that this way. You should use :

void addEventListener( 
  in DOMString type, 
  in EventListener listener, 
  in boolean useCapture 
); 

Documentation right here.

Answer 6

You have three different problems. First of all, values in HTML tags should be quoted! Not doing this can confuse the browser, and may cause some troubles (although it is likely not the case here). Second, you should actually assign a function to the onclick variable, as someone else meantioned. Not only is this the proper way to do it going forward, but it makes things much simpler if you are trying to use local variables in the onclick function. Finally, you can try either addEventListener or jQuery, jQuery has the advantage of a nicer interface.

Oh, and make sure your HTML validates! That could be an issue.