Fix for issue #506 (#525)

bbrala · web-flow · commit e0b1e6185b42 · 2017-04-24T16:04:11.000+02:00
Fixes issue #506 by being a little more conservative in the zIndex check * Update changelog
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,7 @@
 * ContextMenu appears with wrong position ([Issue #502](https://github.com/swisnl/jQuery-contextMenu/issues/502) thanks @apptaro
 * Check if given selected value is a 0, if it is a zero so return it as is. Thanks @Falseee 
 * Events are never trigger when opening a contextMenu right after the other ([Issue #454](https://github.com/swisnl/jQuery-contextMenu/issues/454) thanks @kagant15
+* Accesskey jQuery Modal Dialog not working ([Issue #506](https://github.com/swisnl/jQuery-contextMenu/issues/506) thanks @CiTRO33
 
 ### 2.4.4
 
diff --git a/src/jquery.contextMenu.js b/src/jquery.contextMenu.js
@@ -549,7 +549,7 @@
                 // If targetZIndex is heigher then opt.zIndex dont progress any futher.
                 // This is used to make sure that if you are using a dialog with a input / textarea / contenteditable div
                 // and its above the contextmenu it wont steal keys events
-                if (targetZIndex > opt.zIndex) {
+                if (opt.$menu && parseInt(targetZIndex,10) > parseInt(opt.$menu.css("zIndex"),10)) {
                     return;
                 }
                 switch (e.keyCode) {

Original file line number	Diff line number	Diff line change
`@@ -549,7 +549,7 @@`
`549`	`549`	`// If targetZIndex is heigher then opt.zIndex dont progress any futher.`
`550`	`550`	`// This is used to make sure that if you are using a dialog with a input / textarea / contenteditable div`
`551`	`551`	`// and its above the contextmenu it wont steal keys events`
`552`		`- if (targetZIndex > opt.zIndex) {`
	`552`	`+ if (opt.$menu && parseInt(targetZIndex,10) > parseInt(opt.$menu.css("zIndex"),10)) {`
`553`	`553`	`return;`
`554`	`554`	`}`
`555`	`555`	`switch (e.keyCode) {`