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

mar10 · mar10 · commit 465bdf59d503 · 2017-01-27T23:39:15.000+01:00
Close mar10#110
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");
 		}
 	},

Original file line number	Diff line number	Diff line change
`@@ -372,6 +372,8 @@ $.widget("moogle.contextmenu", {`
`372`	`372`	`$ul = $("<ul/>").appendTo($entryLi);`
`373`	`373`	`$.moogle.contextmenu.createMenuMarkup(entry.children, $ul);`
`374`	`374`	`}`
	`375`	`+ // #110: jQuery UI 1.12: refresh only works when this class is not set:`
	`376`	`+ $entryLi.removeClass("ui-menu-item");`
`375`	`377`	`this.getMenu().menu("refresh");`
`376`	`378`	`}`
`377`	`379`	`},`