Issue 1181 - Change polyfill for IE8 (fix settings page)

html/static/js/ieEventListenerPolyfill.js

-if (typeof Element.prototype.addEventListener != "function")
+if (typeof window.addEventListener != "function")
 {
-  Element.prototype.addEventListener = function(type, handler, capture)
+  window.addEventListener = function(type, handler, capture)
   {
     this.attachEvent("on" + type, handler)
   };
 }
 
-if (typeof Element.prototype.removeEventListener != "function")
+if (typeof window.removeEventListener != "function")
 {
-  Element.prototype.removeEventListener = function(type, handler)
+  window.removeEventListener = function(type, handler)
   {
     this.detachEvent("on" + type, handler)
   };
 }
 
-if (Window.addEventListener != "function")

[Felix Dahlke, 2014/08/08 13:22:33]

Either Window.prototype or window, not Window.

Anyway - we can probably leave this as it was, eh? Doesn't seem to be related to
the IE8 fix, and if we don't use addEventListener on anything but window, what
we had before was fine.

+if (typeof Element.prototype.addEventListener != "function")
 {
-  Window.prototype.addEventListener = Element.prototype.addEventListener;
+  Element.prototype.addEventListener = window.addEventListener;

[Oleksandr, 2014/08/08 14:19:34]

Looks like we've tried every combination now :)

[Felix Dahlke, 2014/08/08 14:21:12]

Yes, I like this one best :)

 }
 
-if (Window.removeEventListener != "function")
+if (typeof Element.prototype.removeEventListener != "function")
 {
-  Window.prototype.removeEventListener = Element.prototype.removeEventListener; 
+  Element.prototype.removeEventListener = window.removeEventListener;
 }