Skip to content

Commit

Permalink
fixing separators to not be clickable - Issue #85
Browse files Browse the repository at this point in the history
  • Loading branch information
rodneyrehm committed Dec 29, 2012
1 parent 2d6c9d6 commit 0b6c31a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,7 @@ $.contextMenu is published under the [MIT license](http://www.opensource.org/lic
* adding class `context-menu-active` to define state on active trigger element - ([Issue 92](https://github.com/medialize/jQuery-contextMenu/issues/92))
* adding [demo for TouchSwipe](http://medialize.github.com/jQuery-contextMenu/demo/trigger-swipe.html) activation
* fixing iOS "click" compatibility problem - ([Issue 83](https://github.com/medialize/jQuery-contextMenu/issues/83))
* fixing separators to not be clickable - ([Issue 85](https://github.com/medialize/jQuery-contextMenu/issues/85))


### 1.5.25 ###
Expand Down
2 changes: 1 addition & 1 deletion src/jquery.contextMenu.js
Original file line number Diff line number Diff line change
Expand Up @@ -713,7 +713,7 @@ var // currently active contextMenu trigger
callback;

// abort if the key is unknown or disabled or is a menu
if (!opt.items[key] || $this.hasClass('disabled') || $this.hasClass('context-menu-submenu')) {
if (!opt.items[key] || $this.is('.disabled, .context-menu-submenu, .context-menu-separator, .not-selectable')) {
return;
}

Expand Down

0 comments on commit 0b6c31a

Please sign in to comment.