From e99974004480d968d3fe37ee57540e72bbc96098 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Sun, 9 Jan 2011 23:18:53 -0800 Subject: fix addEventLister detection for IE9 IE9 added support for addEventListener and it looks like some new events (e.g. DOMContentLoaded) can't be registered via attachEvent, only addEventListener correctly registeres these. So to make sure we are using the latest and greatest we need to look for addEventListener instead of attachEvent in the jqLite feature detection code. --- src/jqLite.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/jqLite.js b/src/jqLite.js index 1bc966eb..0f6bed2d 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -5,12 +5,12 @@ var jqCache = {}, jqName = 'ng-' + new Date().getTime(), jqId = 1, - addEventListenerFn = (window.document.attachEvent ? - function(element, type, fn) {element.attachEvent('on' + type, fn);} : - function(element, type, fn) {element.addEventListener(type, fn, false);}), - removeEventListenerFn = (window.document.detachEvent ? - function(element, type, fn) {element.detachEvent('on' + type, fn); } : - function(element, type, fn) { element.removeEventListener(type, fn, false); }); + addEventListenerFn = (window.document.addEventListener ? + function(element, type, fn) {element.addEventListener(type, fn, false);} : + function(element, type, fn) {element.attachEvent('on' + type, fn);}), + removeEventListenerFn = (window.document.removeEventListener ? + function(element, type, fn) {element.removeEventListener(type, fn, false); } : + function(element, type, fn) {element.detachEvent('on' + type, fn); }); function jqNextId() { return (jqId++); } -- cgit v1.2.3