www

commit 31992363fd189b49432ee850a71467f67a43702f
parent d7f946e5cbe74afc66ea92afbac61beb5936d787
Author: Davide P. Cervone <dpvc@union.edu>
Date:   Sun, 13 Sep 2015 17:44:11 -0400

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.)

Diffstat:
M
unpacked/extensions/MathMenu.js
 | 
10
++++------

1 file changed, 4 insertions(+), 6 deletions(-)
diff --git 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;