Replace window.onmessage with window.addEventListener

[nickdima]

When having multiple bundles that use hot module replacement loaded in the page, window.onmessage got overwritten by the last loaded bundle, making it the only one receiving the message. Changed it with window.addEventListener so it gets handled by all the bundles. See discussion here: webpack-contrib/extract-text-webpack-plugin#30

[jhnns]

👍

[sokra]

https://developer.mozilla.org/en-US/docs/Web/API/EventTarget.addEventListener

In Internet Explorer versions prior to IE 9, you have to use attachEvent rather than the standard addEventListener.

Could you add this?

[nickdima]

I'll look into this. Even if I don't see anyone developing on IE8 or planing to use HMR in production for IE8. :)

[sokra]

It's easy to support, and may be used for hot-testing in different browsers.

[nickdima]

Hot-testing in different browsers sounds like a good case. OK, I'll get down to it.

[akre54]

It just has to look like this: https://github.com/jashkenas/backbone/blob/4927b6e3f013821a5c94a9e8f6e3d5c3acdeaca8/backbone.js#L1474-L1477

var addEventListener = window.addEventListener || function (eventName, listener) {
  return attachEvent('on' + eventName, listener);
};

addEventListener('message', fn);

[nickdima]

Done, thanks @akre54