Fix setEntry({...}) for jQuery UI 1.12

Close #110
mar10 · Jan 27, 2017 · 465bdf5 · 465bdf5
1 parent 761ceee
commit 465bdf5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,6 @@
 # 1.13.1-0 / Unreleased
  * [BUGFIX] #109 Fix `setEntry(title)` for titles containing icons
+ * [BUGFIX] #110 Fix `setEntry({...})` for jQuery UI 1.12
 
 # 1.13.0 / 2016-08-21
 

diff --git a/jquery.ui-contextmenu.js b/jquery.ui-contextmenu.js
@@ -372,6 +372,8 @@ $.widget("moogle.contextmenu", {
 				$ul = $("<ul/>").appendTo($entryLi);
 				$.moogle.contextmenu.createMenuMarkup(entry.children, $ul);
 			}
+			// #110: jQuery UI 1.12: refresh only works when this class is not set:
+			$entryLi.removeClass("ui-menu-item");
 			this.getMenu().menu("refresh");
 		}
 	},