Allow to show/hide separators

mar10 · mar10 · commit 599ea7c3fcb0 · 2015-09-01T22:27:32.000+02:00
Close mar10#98
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 # 1.10.1-0 / Unreleased
 
 * [FEATURE] #94 Add `main` field to package.json
+* [CHANGE] #98 Allow to show/hide separators
 
 # 1.10.0 / 2015-06-27
 
diff --git a/README.md b/README.md
@@ -87,6 +87,7 @@ $(document).contextmenu({
 });
 ```
 **Note:** only one contextmenu widget instance can be bound to one element.
+See the *Howto* below for a solution to this problem.
 
 The `menu` options may contain a (nested) array of entry defiitions.
 Following a list of available properties:
diff --git a/jquery.ui-contextmenu.js b/jquery.ui-contextmenu.js
@@ -377,13 +377,14 @@ $.extend($.moogle.contextmenu, {
 	createEntryMarkup: function(entry, $parentLi) {
 		var $a = null;
 
+		$parentLi.attr("data-command", entry.cmd);
+
 		if ( !/[^\-\u2014\u2013\s]/.test( entry.title ) ) {
 			// hyphen, em dash, en dash: separator as defined by UI Menu 1.10
 			$parentLi.text(entry.title);
 		} else {
 			if ( isLTE110 ) {
 				// jQuery UI Menu 1.10 or before required an `<a>` tag
-				$parentLi.attr("data-command", entry.cmd);
 				$a = $("<a/>", {
 						html: "" + entry.title,
 						href: "#"
@@ -395,9 +396,7 @@ $.extend($.moogle.contextmenu, {
 
 			} else {
 				// jQuery UI Menu 1.11+ preferes to avoid `<a>` tags
-				$parentLi
-					.attr("data-command", entry.cmd)
-					.html("" + entry.title);
+				$parentLi.html("" + entry.title);
 				if ( $.isFunction(entry.action) ) {
 					$parentLi.data("actionHandler", entry.action);
 				}
