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

html/static/js/ieEventListenerPolyfill.js

Index: html/static/js/ieEventListenerPolyfill.js
===================================================================
--- a/html/static/js/ieEventListenerPolyfill.js
+++ b/html/static/js/ieEventListenerPolyfill.js
@@ -1,15 +1,25 @@
-if (typeof window.addEventListener != "function")
+if (typeof Element.prototype.addEventListener != "function")
 {
-  window.addEventListener = function(type, handler, capture)
+  Element.prototype.addEventListener = function(type, handler, capture)
   {
     this.attachEvent("on" + type, handler)
   };
 }
 
-if (typeof window.removeEventListener != "function")
+if (typeof Element.prototype.removeEventListener != "function")
 {
-  window.removeEventListener = function(type, handler)
+  Element.prototype.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.

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