New option autoTrigger: true can be set to false to prevent opening menus on browser's contextmenu event

Closed mar10#25

Author: mar10

CHANGELOG.md

@@ -1,7 +1,9 @@
 # 1.3.0 / unreleased
 
 * [FEATURE] New optional parameter open(..., extraData).
-* [FEATURE] New option `autoTrigger: true` can be set to `false` to prevent opening on browser's `contextmenu` event.
+* [FEATURE] New option `autoTrigger: true` can be set to `false` to prevent opening menus on browser's `contextmenu` event 
+  (if you want to use the `open()` method instead).
+* [FEATURE] New option `preventContextMenuForPopup`to prevent opening the browser's system context menu on menu entries.
 * [CHANGE] `setEntry()` and `replaceMenu()` now allow to define titles with HTML markup.
 
 # 1.2.4 / 2013-12-25

README.md

@@ -205,6 +205,12 @@ $(document).contextmenu({
             return {my: "left top", at: "left bottom", of: ui.target};
         }, ...</pre>
 </dd>
+<dt>preventContextMenuForPopup</dt>
+<dd>
+    Type: <code>Boolean</code>, default: <code>false</code><br>
+    Prevent that a right click inside an open popup menu will open the browser's 
+    system context menu.
+</dd>
 <dt>preventSelect</dt>
 <dd>
     Type: <code>Boolean</code>, default: <code>false</code><br>

demo/index.html

@@ -78,6 +78,7 @@
 
 	$(document).contextmenu({
 		delegate: ".hasmenu",
+		preventContextMenuForPopup: true,
 		preventSelect: true,
 		taphold: true,
 		menu: [

jquery.ui-contextmenu.js

@@ -26,6 +26,7 @@
 			ignoreParentSelect: true, // Don't trigger 'select' for sub-menu parents
 			menu: null,           // selector or jQuery pointing to <UL>, or a definition hash
 			position: null,       // popup positon
+			preventContextMenuForPopup: false, // prevent opening the browser's system context menu on menu entries
 			preventSelect: false, // disable text selection of target
 			show: { effect: "slideDown", duration: "fast"},
 			taphold: false,       // open menu on taphold events (requires external plugins)
@@ -212,6 +213,11 @@
 				}).position(posOption)
 				.hide(); // hide again, so we can apply nice effects
 
+			if( opts.preventContextMenuForPopup ) {
+				this.$menu.bind("contextmenu" + this.eventNamespace, function(event){
+					event.preventDefault();
+				});
+			}
 			this._show(this.$menu, this.options.show, function(){
 				self._trigger.call(self, "open", event, ui);
 			});
@@ -226,6 +232,8 @@
 				.unbind("mousedown" + this.eventNamespace)
 				.unbind("touchstart" + this.eventNamespace)
 				.unbind("keydown" + this.eventNamespace);
+			this.$menu
+				.unbind("contextmenu" + this.eventNamespace);
 			self.currentTarget = null; // issue #44 after hide animation is too late
 
 			this._hide(this.$menu, hideOpts, function() {

jquery.ui-contextmenu.min.js

@@ -4,50 +4,50 @@
   "title": "jQuery UI context menu plugin",
   "description": "Turn a jQuery UI Menu widget into a contextmenu.",
   "keywords": [
-    "context-menu",
-    "contextmenu",
-    "delegate",
-    "jquery-ui-menu",
-    "menu",
-    "navigation",
-    "popup",
-    "right-click",
-    "right-click-menu"
+	"context-menu",
+	"contextmenu",
+	"delegate",
+	"jquery-ui-menu",
+	"menu",
+	"navigation",
+	"popup",
+	"right-click",
+	"right-click-menu"
   ],
   "author": {
-    "name": "Martin Wendt",
-    "email": "jquery@wwwendt.de",
-    "url": "http://careers.stackoverflow.com/martin-wendt"
+	"name": "Martin Wendt",
+	"email": "jquery@wwwendt.de",
+	"url": "http://careers.stackoverflow.com/martin-wendt"
   },
   "repository": {
-    "type": "git",
-    "url": "https://github.com/mar10/jquery-ui-contextmenu"
+	"type": "git",
+	"url": "https://github.com/mar10/jquery-ui-contextmenu"
   },
   "homepage": "https://github.com/mar10/jquery-ui-contextmenu",
   "demo": "http://wwwendt.de/tech/demo/jquery-contextmenu/demo/",
   "docs": "https://github.com/mar10/jquery-ui-contextmenu/blob/master/README.md",
   "bugs": "https://github.com/mar10/jquery-ui-contextmenu/issues",
   "licenses": [
-    {
-      "type": "MIT",
-      "url": "https://github.com/mar10/jquery-ui-contextmenu/blob/master/MIT-LICENSE.txt"
-    }
+	{
+	  "type": "MIT",
+	  "url": "https://github.com/mar10/jquery-ui-contextmenu/blob/master/MIT-LICENSE.txt"
+	}
   ],
   "dependencies": {
-    "jquery": ">=1.7"
+	"jquery": ">=1.7"
   },
   "devDependencies": {
-    "grunt": "~0.4.1",
-    "grunt-contrib-concat": "~0.1.3",
-    "grunt-contrib-connect": "~0.3.0",
-    "grunt-contrib-jshint": "~0.3.0",
-    "grunt-contrib-qunit": "~0.2.0",
-    "grunt-contrib-uglify": "~0.2.0",
-    "grunt-exec": "~0.4.0",
-    "grunt-saucelabs": "~4.1.2",
-    "grunt-contrib-watch": "~0.5.3"
+	"grunt": "~0.4.1",
+	"grunt-contrib-concat": "~0.1.3",
+	"grunt-contrib-connect": "~0.3.0",
+	"grunt-contrib-jshint": "~0.3.0",
+	"grunt-contrib-qunit": "~0.2.0",
+	"grunt-contrib-uglify": "~0.2.0",
+	"grunt-exec": "~0.4.0",
+	"grunt-saucelabs": "~4.1.2",
+	"grunt-contrib-watch": "~0.5.3"
   },
   "scripts": {
-    "test": "grunt ci --verbose"
+	"test": "grunt ci --verbose"
   }
 }