New optional parameter open(..., extraData)

closed mar10#42

Author: mar10

CHANGELOG.md

@@ -1,6 +1,6 @@
-# 1.2.5 / unreleased
+# 1.3.0 / unreleased
 
-* 
+* [FEATURE] New optional parameter open(..., extraData)
 
 # 1.2.4 / 2013-12-25
 

README.md

@@ -202,10 +202,11 @@ structure (see [jQueryUI menu] for details):
 <dd>
     Return true if popup is visible.
 </dd>
-<dt>open(target)</dt>
+<dt>open(target[, extraData])</dt>
 <dd>
     Open context menu on a specific target (target must match the options.delegate filter).<br>
-    Call like <code>$(...).contextmenu("open", target);</code>.
+    Call like <code>$(...).contextmenu("open", target[, extraData]);</code>.
+    Optional `extraData` will be available in eventHandlers as <code>ui.extraData</code>.
 </dd>
 <dt>replaceMenu(menu)</dt>
 <dd>

demo/index.html

@@ -107,7 +107,9 @@
 		// Implement the beforeOpen callback to dynamically change the entries
 		beforeOpen: function(event, ui) {
 			var $menu = ui.menu,
-				$target = ui.target;
+				$target = ui.target,
+				extraData = ui.extraData; // passed when menu was opened by call to open()
+			// console.log("beforeOpen", event, ui, event.originalEvent.type);
 			$(document)
 //				.contextmenu("replaceMenu", [{title: "aaa"}, {title: "bbb"}])
 //				.contextmenu("replaceMenu", "#options2")
@@ -158,7 +160,7 @@
 
 	$("#triggerPopup").click(function(){
 		// Trigger popup menu on the first target element
-		$(document).contextmenu("open", $(".hasmenu:first"));
+		$(document).contextmenu("open", $(".hasmenu:first"), {foo: "bar"});
 		setTimeout(function(){
 			$(document).contextmenu("close");
 		}, 2000);

jquery.ui-contextmenu.js

@@ -18,7 +18,7 @@
 
 
 	$.widget("moogle.contextmenu", {
-		version: "1.2.5-1",
+		version: "1.3.0-1",
 		options: {
 			delegate: null,       // selector
 			hide: { effect: "fadeOut", duration: "fast"},
@@ -160,7 +160,7 @@
 			var opts = this.options,
 				posOption = opts.position,
 				self = this,
-				ui = {menu: this.$menu, target: $(event.target)};
+				ui = {menu: this.$menu, target: $(event.target), extraData: event.extraData};
 			this.currentTarget = event.target;
 			// Prevent browser from opening the system context menu
 			event.preventDefault();
@@ -248,7 +248,6 @@
 		enableEntry: function(cmd, flag){
 			this._getMenuEntry(cmd, true).toggleClass("ui-state-disabled", (flag === false));
 		},
-		/** Redefine the whole menu. */
 		/** Return Menu element (UL). */
 		getMenu: function(){
 			return this.$menu;
@@ -258,10 +257,13 @@
 //            return this.$menu && this.$menu.is(":visible");
 			return !!this.$menu && !!this.currentTarget;
 		},
-		/** Open context menu on a specific target (must match options.delegate) */
-		open: function(target){
+		/** Open context menu on a specific target (must match options.delegate)
+		 *  Optional `extraData` is passed to event handlers as `ui.extraData`.
+		 */
+		open: function(target, extraData){
 			// Fake a 'contextmenu' event
-			var e = jQuery.Event("contextmenu", {target: target.get(0)});
+			extraData = extraData || {};
+			var e = jQuery.Event("contextmenu", {target: target.get(0), extraData: extraData});
 			return this.element.trigger(e);
 		},
 		/** Replace the menu altogether. */

jquery.ui-contextmenu.min.js

@@ -1,6 +1,6 @@
 {
   "name": "ui-contextmenu",
-  "version": "1.2.5-1",
+  "version": "1.3.0-1",
   "title": "jQuery UI context menu plugin",
   "description": "Turn a jQuery UI Menu widget into a contextmenu.",
   "keywords": [

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "ui-contextmenu",
-	"version": "1.2.5-1",
+	"version": "1.3.0-1",
 
 	"title": "jQuery UI context menu plugin",
 	"description": "Turn a jQuery UI Menu widget into a contextmenu.",