Postitioning of menu with appendTo

mrMarco · mrMarco · commit ee0a1d1d22ed · 2015-12-27T00:22:08.000+01:00
My last solution wasn't browser-independant. now it is.
diff --git a/src/jquery.contextMenu.js b/src/jquery.contextMenu.js
@@ -333,9 +333,10 @@
                     if (showMenu) {
                         // show menu
 		                var menuContainer = (e.data.appendTo === null ? $('body') : $(e.data.appendTo));
+		                var srcElement = e.target || e.srcElement || e.originalTarget;
 		                op.show.call($this, e.data,
-                					 $(e.srcElement).offset().left - menuContainer.offset().left + e.offsetX,
-                					 $(e.srcElement).offset().top - menuContainer.offset().top + e.offsetY);
+                					 $(srcElement).offset().left - menuContainer.offset().left + e.offsetX,
+                					 $(srcElement).offset().top - menuContainer.offset().top + e.offsetY);
                     }
                 }
             },

Original file line number	Diff line number	Diff line change
`@@ -333,9 +333,10 @@`
`333`	`333`	`if (showMenu) {`
`334`	`334`	`// show menu`
`335`	`335`	`var menuContainer = (e.data.appendTo === null ? $('body') : $(e.data.appendTo));`
	`336`	`+ var srcElement = e.target \|\| e.srcElement \|\| e.originalTarget;`
`336`	`337`	`op.show.call($this, e.data,`
`337`		`- $(e.srcElement).offset().left - menuContainer.offset().left + e.offsetX,`
`338`		`- $(e.srcElement).offset().top - menuContainer.offset().top + e.offsetY);`
	`338`	`+ $(srcElement).offset().left - menuContainer.offset().left + e.offsetX,`
	`339`	`+ $(srcElement).offset().top - menuContainer.offset().top + e.offsetY);`
`339`	`340`	`}`
`340`	`341`	`}`
`341`	`342`	`},`