Close mar10#1

Author: mar10

README.md

@@ -79,12 +79,19 @@ structure (see [jQueryUI menu] for details):
     Type: <code>String</code><br>
     A selector to filter the elements that trigger the context menu.    
 </dd>
+<dt>ignoreParentSelect</dt>
+<dd>
+    Type: <code>Boolean</code>, default: <code>true</code><br>
+    If <code>true</code>, a click on a menu item that contains a sub-menu, will <em>not</em>
+    trigger the <code>select</code> event.
+</dd>
 <dt>menu</dt>
 <dd>
     Type: <code>String | jQuery | function</code><br>
     jQuery object or selector (or function returning such) of HTML markup that defines the context menu
     structure (see [jQueryUI menu] for details).
 </dd>
+
 </dl>
 
 

jquery.contextmenu.js

@@ -18,6 +18,7 @@
 		version: "0.0.1",
 		options: {
 			delegate: "[data-menu]",  // selector
+			ignoreParentSelect: true, // Don't trigger 'select' for sub-menu parents 
 			menu: null,      // selector or jQuery or a function returning such
 			taphold: 2000, // open menu after 2000 ms long touch
 			// Events:
@@ -100,9 +101,12 @@
 					select: function(event, ui){
 					    // Also pass the target that the menu was triggered on:
 					    event.relatedTarget = openEvent.target;
-						if( self._trigger.call(self, "select", event, ui) !== false ){
-							self._closeMenu.call(self);
-						}
+					    var isParent = (ui.item.has(">a[aria-haspopup='true']").length > 0);
+					    if( !isParent || !self.options.ignoreParentSelect){
+	                        if( self._trigger.call(self, "select", event, ui) !== false ){
+	                            self._closeMenu.call(self);
+	                        }
+					    }
 					}
 				});
 			// Register global event handlers that close the dropdown-menu
@@ -117,7 +121,7 @@
 				}
 			});
 			$menu
-			    .show() // required to fix positioning error (issue #)
+			    .show() // required to fix positioning error (issue #3)
 				.css({
 					position: "absolute",
 					left: 0,

sample-widget.html

@@ -27,6 +27,7 @@
 $(function(){
 	$("#container").contextmenu({
 		delegate: ".hasmenu",
+//		ignoreParentSelect: false,
 		menu: "#options",
 		focus: function(event, ui) {
 			var menuId = ui.item.find(">a").attr("href");
@@ -36,7 +37,7 @@
 			$("#info").text("");
 		},
         beforeOpen: function(event) {
-//          alert("beforeopen on " + $(event.relatedTarget).text());
+          alert("beforeopen on " + $(event.relatedTarget).text());
         },
         open: function(event, ui) {
 //          alert("open on " + $(event.relatedTarget).text());