www

commit c7be71b554732713a1442c5bcb8220e529da772c
parent 2afb80176c0ed5be08efe5babf9b0dbd4835fbf5
Author: Davide P. Cervone <dpvc@union.edu>
Date:   Sat,  8 Feb 2014 15:55:21 -0500

Put checking for screenX into try...catch for IE8 (grr).  Resolves issue #348.

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

1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/unpacked/extensions/MathMenu.js b/unpacked/extensions/MathMenu.js
@@ -761,8 +761,10 @@
         var H = (w.outerHeight-w.innerHeight)||30, W = (w.outerWidth-w.innerWidth)||30, x, y;
         W = Math.max(100,Math.min(Math.floor(.5*screen.width),table.offsetWidth+W+25));
         H = Math.max(40,Math.min(Math.floor(.5*screen.height),table.offsetHeight+H+25));
+        if (MENU.prototype.msieHeightBug) {H += 35}; // for title bar in XP
         w.resizeTo(W,H);
-        if (event && event.screenX != null) {
+        var X; try {X = event.screenX} catch (e) {}; // IE8 throws an error accessing screenX
+        if (event && X != null) {
           x = Math.max(0,Math.min(event.screenX-Math.floor(W/2), screen.width-W-20));
           y = Math.max(0,Math.min(event.screenY-Math.floor(H/2), screen.height-H-20));
           w.moveTo(x,y);
@@ -955,9 +957,11 @@
       var isIE8 = browser.versionAtLeast("8.0") && document.documentMode > 7;
       MENU.Augment({
         margin: 20,
-        msieBackgroundBug: (document.documentMode < 9),
+        msieBackgroundBug: ((document.documentMode||0) < 9),
         msieFixedPositionBug: (quirks || !isIE8),
-        msieAboutBug: quirks
+        msieAboutBug: quirks,
+        msieHeightBug: ((document.documentMode||0) < 9)
+           // height of window doesn't include title bar in XP
       });
       if (isIE9) {
         delete CONFIG.styles["#MathJax_About"].filter;