Add new menu option isHeader for category headers

Close #100
mar10 · May 14, 2016 · 733a4ff · 733a4ff · maasha · May 17, 2016
1 parent 7cf32e3
commit 733a4ff
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,8 @@
 # 1.11.1-0 / Unreleased
 
  * [FEATURE] Support jQuery UI 1.12 (wrap menu items in separate `<div>` tags)
- * Add jQuery Ui 1.12 sample with new 'Base' theme
+ * [FEATURE] #100 new menu option `isHeader` for category headers.
+ * Add jQuery UI 1.12 sample with new 'Base' theme
 
 # 1.11.0 / 2015-09-10
 

diff --git a/demo/index-1-12.html b/demo/index-1-12.html
@@ -82,6 +82,7 @@
 		preventSelect: true,
 		taphold: true,
 		menu: [
+			{title: "Menu Header", isHeader: true},
 			{title: "Cut <kbd>Ctrl+X</kbd>", cmd: "cut", uiIcon: "ui-icon-scissors"},
 			{title: "Copy <kbd>Ctrl+C</kbd>", cmd: "copy", uiIcon: "ui-icon-copy"},
 			{title: "Paste <kbd>Ctrl+V</kbd>", cmd: "paste", uiIcon: "ui-icon-clipboard", disabled: true },

diff --git a/demo/teaser-1.11.png b/demo/teaser-1.11.png
diff --git a/demo/teaser.png b/demo/teaser.png
diff --git a/jquery.ui-contextmenu.js b/jquery.ui-contextmenu.js
@@ -159,6 +159,7 @@ $.widget("moogle.contextmenu", {
 			.addClass(opts.addClass)
 			// Create a menu instance that delegates events to our widget
 			.menu($.extend(true, {}, opts.uiMenuOptions, {
+				items: "> :not(.ui-widget-header)",
 				blur: $.proxy(opts.blur, this),
 				create: $.proxy(opts.createMenu, this),
 				focus: $.proxy(opts.focus, this),
@@ -422,6 +423,9 @@ $.extend($.moogle.contextmenu, {
 			if ( entry.disabled ) {
 				$parentLi.addClass("ui-state-disabled");
 			}
+			if ( entry.isHeader ) {
+				$parentLi.addClass("ui-widget-header");
+			}
 			if ( entry.addClass ) {
 				$parentLi.addClass(entry.addClass);
 			}