From 31992363fd189b49432ee850a71467f67a43702f Mon Sep 17 00:00:00 2001 From: "Davide P. Cervone" Date: Sun, 13 Sep 2015 17:44:11 -0400 Subject: [PATCH] Fix problems found during the running of the testsuite. (First, make sure event is non-null, but I think that was already the case. So we don't need the later tests for it. Then use clientX and clienY even if they are zero.) --- unpacked/extensions/MathMenu.js | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/unpacked/extensions/MathMenu.js b/unpacked/extensions/MathMenu.js index c3105400d..9e47031c6 100644 --- a/unpacked/extensions/MathMenu.js +++ b/unpacked/extensions/MathMenu.js @@ -282,7 +282,7 @@ * Display the menu */ Post: function (event,parent,forceLTR) { - if (!event) {event = window.event}; + if (!event) {event = window.event||{}} var div = document.getElementById("MathJax_MenuFrame"); if (!div) { div = MENU.Background(this); @@ -312,10 +312,8 @@ div.appendChild(menu); this.posted = true; if (menu.offsetWidth) menu.style.width = (menu.offsetWidth+2) + "px"; - if (event) { - var x = event.pageX, y = event.pageY; - } - if (!x && !y && event && event.clientX && event.clientY) { + var x = event.pageX, y = event.pageY; + if (!x && !y && "clientX" in event) { x = event.clientX + document.body.scrollLeft + document.documentElement.scrollLeft; y = event.clientY + document.body.scrollTop + document.documentElement.scrollTop; } @@ -331,7 +329,7 @@ if (x + menu.offsetWidth > document.body.offsetWidth - this.margin) {x = document.body.offsetWidth - menu.offsetWidth - this.margin} if (MENU.isMobile) {x = Math.max(5,x-Math.floor(menu.offsetWidth/2)); y -= 20} - if (event) {MENU.skipUp = event.isContextMenu;} + MENU.skipUp = event.isContextMenu; } else { var side = "left", mw = parent.offsetWidth; x = (MENU.isMobile ? 30 : mw - 2); y = 0;