(fix) Use proper event object for handlers in IE

feimosi · feimosi · commit f1b54847d6c1 · 2017-03-11T15:07:59.000+01:00
`event` and `event.target` are not provided in IE8.
This fixes the bugs with not hiding the overlay on background click
and not working keyboard shortcuts.
diff --git a/src/baguetteBox.js b/src/baguetteBox.js
@@ -663,7 +663,12 @@
             element.addEventListener(event, callback, useCapture);
         } else {
             // IE8 fallback
-            element.attachEvent('on' + event, callback);
+            element.attachEvent('on' + event, function(event) {
+                // `event` and `event.target` are not provided in IE8
+                event = event || window.event;
+                event.target = event.target || event.srcElement;
+                callback(event);
+            });
         }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -663,7 +663,12 @@`
`663`	`663`	`element.addEventListener(event, callback, useCapture);`
`664`	`664`	`} else {`
`665`	`665`	`// IE8 fallback`
`666`		`- element.attachEvent('on' + event, callback);`
	`666`	`+ element.attachEvent('on' + event, function(event) {`
	`667`	+ // `event` and `event.target` are not provided in IE8
	`668`	`+ event = event \|\| window.event;`
	`669`	`+ event.target = event.target \|\| event.srcElement;`
	`670`	`+ callback(event);`
	`671`	`+ });`
`667`	`672`	`}`
`668`	`673`	`}`
`669`	`674`