fix: add way to stop propagation when an item is clicked

Closes #105
Templarian · Oct 1, 2017 · e441add · e441add
1 parent bf7ec94
commit e441add
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 3 deletions.
diff --git a/contextMenu.js b/contextMenu.js
@@ -196,17 +196,24 @@ angular.module('ui.bootstrap.contextMenu', [])
                 if($event.which == 1) {
                   $event.preventDefault();
                   $scope.$apply(function () {
-                    $(event.currentTarget).removeClass('context');
-                    removeAllContextMenus();
 
+                    var cleanupFunction = function () {
+                      $(event.currentTarget).removeClass('context');
+                      removeAllContextMenus();
+                    };
                     var clickFunction = angular.isFunction(item.click)
                       ? item.click
                       : (angular.isFunction(item[1])
                           ? item[1]
                           : null);
 
                     if (clickFunction) {
-                      clickFunction.call($scope, $scope, event, modelValue, text, $li);
+                      var res = clickFunction.call($scope, $scope, event, modelValue, text, $li);
+                      if(res === undefined || res) {
+                        cleanupFunction();
+                      }
+                    } else {
+                      cleanupFunction();
                     }
                   });
                 }

diff --git a/demo/demo.js b/demo/demo.js
@@ -42,6 +42,14 @@ angular.module('app', ['ui.bootstrap.contextMenu'])
                 alert('A new implementation based on objects');
               }
             },
+            {
+              text: 'Clicking this does not close the context menu',
+              click: function () {
+                // Returning false or any false-y value EXCEPT undefined will stop
+                // the context menu from closing.
+                return false;
+              }
+            },
             {
               text: 'Object-based with Submenu',
               click: function() {