Fix a bug with IE regarding window.scrollX/Y not being supported

Signed-off-by: Yannick Gagnon <yannick.gagnon@lemieuxbedard.com>
moobilejs · Nov 9, 2016 · 22b1694 · 22b1694
1 parent d29908d
commit 22b1694
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/dist/js/moobile.js b/dist/js/moobile.js
@@ -7758,8 +7758,9 @@ Element.defineCustomEvent('tap', {
 	condition: function(e) {
 
 		if (tapValid) {
-
-			var element = tapTouch ? document.elementFromPoint(tapTouch.pageX - window.scrollX, tapTouch.pageY - window.scrollY) : null;
+			var offsetX = window.scrollX === undefined ? window.pageXOffset : window.scrollX,
+				offsetY = window.scrollY === undefined ? window.pageYOffset : window.scrollY,
+				element = tapTouch ? document.elementFromPoint(tapTouch.pageX - offsetX, tapTouch.pageY - offsetY) : null;
 			if (element) {
 				return this === element || this.contains(element);
 			}

diff --git a/dist/js/moobile.min.js b/dist/js/moobile.min.js
diff --git a/src/event/tap.js b/src/event/tap.js
@@ -19,8 +19,9 @@ Element.defineCustomEvent('tap', {
 	condition: function(e) {
 
 		if (tapValid) {
-
-			var element = tapTouch ? document.elementFromPoint(tapTouch.pageX - window.scrollX, tapTouch.pageY - window.scrollY) : null;
+			var offsetX = window.scrollX === undefined ? window.pageXOffset : window.scrollX,
+				offsetY = window.scrollY === undefined ? window.pageYOffset : window.scrollY,
+				element = tapTouch ? document.elementFromPoint(tapTouch.pageX - offsetX, tapTouch.pageY - offsetY) : null;
 			if (element) {
 				return this === element || this.contains(element);
 			}