feature(js): requiresConfirmation now returns false if not confirmed

fixes #9240
Elgg · Mar 23, 2016 · cac5c0f · cac5c0f
1 parent b96c736
commit cac5c0f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/js/lib/ui.js b/js/lib/ui.js
@@ -290,15 +290,15 @@ elgg.ui.initHoverMenu = function(parent) {
 };
 
 /**
- * Calls a confirm() and prevents default if denied.
+ * Calls a confirm() and returns false if denied.
  *
  * @param {Object} e
  * @return void
  */
 elgg.ui.requiresConfirmation = function(e) {
 	var confirmText = $(this).data('confirm') || elgg.echo('question:areyousure');
 	if (!confirm(confirmText)) {
-		e.preventDefault();
+		return false;
 	}
 };