Comments on: Image onLoad Event + JavaScript Issue with Internet Explorer

By: viaria

viaria — Fri, 10 Jun 2011 14:25:11 +0000

good solution thanks, just for you to know the corner copy button of second black box is not working. chrome win7

By: Jim

Jim — Mon, 14 Feb 2011 20:35:53 +0000

Just ran across this. Thanks for the solution.

By: alec

alec — Mon, 05 Jul 2010 21:52:11 +0000

There is nothing IE-related or even remotely surprising about this. You need to be aware of the fact that JS can be multi-threaded, and that all requests for external resources are asynchronous (non-blocking). By setting the src property you are in effect spawning a new thread to go get that image, and attempting to set handlers or write inline code around a thread that is already running is just asking for trouble.

You wouldn’t try to set event handlers on an XmlHttpRequest after calling send() would you? The principal is exactly the same.

By: David Walsh

David Walsh — Fri, 11 Jun 2010 13:36:59 +0000

@Adardesign: That’s not a terrible solution but you’re preventing caching instead of modifying your code. Not a good trade off.

By: Adardesign

Adardesign — Fri, 11 Jun 2010 13:28:57 +0000

I had the same issue lately, I added a dynamic parameter to the image src like
path/to/img.jpg+ “?” + new Date().getTime();

This makes sure it doesn’t take the image from cache since it has a new parameter value each time.

Your solution solves the problem with a much clearer and more efficient way.

Thank

By: Richard

Richard — Fri, 11 Jun 2010 05:47:48 +0000

Brilliant work, I had the same issue some time ago.

By: Nickolas Simard

Nickolas Simard — Thu, 10 Jun 2010 20:31:11 +0000

IE is plain crazy sometimes… ok, ALL the time!

By: Marcel M.

Marcel M. — Thu, 10 Jun 2010 19:57:04 +0000

For this example it would add more code, but you can check if a img element has been loaded by its “complete” attribute.

if(img.complete) imgLoaded(); else img.addEvent('load', imgLoaded);

By: David Walsh

David Walsh — Thu, 10 Jun 2010 18:13:24 +0000

@Kendall: Yep, I’m upgrading a lightbox.

By: Tim

Tim — Thu, 10 Jun 2010 18:04:49 +0000

Great pick up! I ran into this issue (not knowing until now) just last week, I moved on to another project but I am now going to go back and revisit that code.

Thanks!