Fixed event bubbling handling for empty space in Share widget and Sli…

…de out tab.
wet-boew · Jul 3, 2012 · 19bc3f8 · 19bc3f8
1 parent 98f13e9
commit 19bc3f8
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 5 deletions.
diff --git a/build/js/pe-ap-min.js b/build/js/pe-ap-min.js
diff --git a/src/js/workers/share.js b/src/js/workers/share.js
@@ -205,8 +205,12 @@
 				});
 
 				// Close the popup menu on any click away from the links
-				elm.on("click touchstart", function (e) {
-					return false;
+				elm.on("click vclick", function (event) {
+					if (event.stopPropagation) {
+						event.stopPropagation();
+					} else {
+						event.cancelBubble = true;
+					}
 				});
 				$(document).on("click touchstart", function (e) {
 					if ($popup.attr('aria-hidden') === 'false') {

diff --git a/src/js/workers/slideout.js b/src/js/workers/slideout.js
@@ -238,8 +238,12 @@
 			elm.on("keydown", keyhandler);
 
 			// Close slideout if clicking outside of the slideout area
-			elm.on("click touchstart", function (e) {
-				return false;
+			elm.on("click vclick", function (event) {
+				if (event.stopPropagation) {
+					event.stopPropagation();
+				} else {
+					event.cancelBubble = true;
+				}
 			});
 			$(document).on("click touchstart", function () {
 				if (opened) {