views:

1596

answers:

4

I need some javascript code that dynamically adds an img tag to a div, and the img tag needs onmouseover and onmouseout handlers.

I have it working on Firefox. But it doesn't quite work on IE. On IE, the img tag is added, but the onmouseover and onmouseout handlers are not active.

Here's the code:

<body>  
 <div id='putImageHere' />  

 <script type='text/javascript'>
  var node = document.getElementById('putImageHere');
  var img = document.createElement('img');
  img.setAttribute('src', 'http://sstatic.net/so/img/logo.png');
  node.appendChild(img);

  // first attempt, which works on Firefox but not IE
  img.setAttribute('onmouseover', "alert('mouseover')");
  img.setAttribute('onmouseout', "alert('mouseout')");

  // second attempt, which I thought would work on IE but doesn't
  img.addEventListener('mouseover', function() { alert('mouseover') }, false);
  img.addEventListener('mouseout', function() { alert('mouseout') }, false);
 </script>  
</body>
A: 

Turns out IE doesn't support the addEventListener method. You can check a workaround here.

Anyway, why don't you use jQuery? It almost overrides all the compatibility issues, has a handful of extras and it totally rocks.

metrobalderas
Thanks! Using img.attachEvent works on IE.There is a reason why I can't use jquery, but my reason isn't applicable to about 99.99% of the people reading this, so I won't go into it.
Mike W
+4  A: 
if (img.addEventListener) {
    img.addEventListener('mouseover', function() {}, false);
    img.addEventListener('mouseout', function() {}, false);
} else { // IE
    img.attachEvent('onmouseover', function() {});
    img.attachEvent('onmouseout', function() {});
}

look into using one of the many popular javascript libraries (jquery, prototype, etc). they hide browser inconsistencies so you don't need to worry about writing code like above.

ob
+2  A: 
img.setAttribute('src', 'http://sstatic.net/so/img/logo.png');

Don't use setAttribute on an HTMLElement, at all. There are bugs in it in Internet Explorer and it doesn't get you anything compared to the more readable DOM Level 1 HTML alternative:

img.src= 'http://sstatic.net/so/img/logo.png';

The essential problem with setAttribute on IE is that it works the same as normal property access, even when the property has a different name or type to the attribute. So this:

img.setAttribute('onmouseover', "alert('mouseover')");

is the same as saying:

img.onmouseover= "alert('mouseover')";

which makes no sense: the onmouseover property is, like all event handlers, supposed to be a function, not a string. Assign a function instead:

img.onmouseover= function() {
    alert('mouseover');
};

and then you've also got rid the nasty eval-style practice of putting code inside a string. Hooray!

As for addEventListener, you can use it if you add a fallback attachEvent call for IE (which has its own crazy event listener system). But in the common case where you only ever have one listener for each element it is much easier to stick with the old-school supported-by-every-browser onsomething-assigning event handlers.

bobince
+1 for explaining clearly why `setAttribute` doesn't work in IE
Tim Goodman
A: 

Ob's answer shows the correct method for attaching event listeners without using a JavaScript library (and was first), but I would like to add the following:

onmouseover/out should never be attached as attributes. Basically, you're gone through all the trouble of not doing

just to do it in your JavaScript. Event handlering callbacks should always be attached as event listeners via attachEvent for IE, and addEventListener for pretty much everyone else.

Justin Johnson