JavaScript脚本在Firefox中不起作用

Question

我有一个旧功能，该功能缺少Mozilla / Firefox的行，因此JavaScript无法在其中正常运行。 该功能跟踪鼠标坐标，以便我可以定位窗口。

如何使代码在Firefox中也能正常工作？

Xoffset = -60; // modify these values to ...
Yoffset = 20; // change the popup position.
var old, skn, iex = (document.all),
    yyy = -1000;

var ns4 = document.layers
var ns6 = document.getElementById && !document.all
var ie4 = document.all

if (ns4) skn = document.dek
else if (ns6) skn = document.getElementById("dek").style
else if (ie4) skn = document.all.dek.style
if (ns4) document.captureEvents(Event.MOUSEMOVE);
else {
  skn.visibility = "visible"
  skn.display = "none"
}
document.onmousemove = get_mouse;


function popup(msg, bak) {
  var content = 
      "<TABLE  WIDTH=150 BORDER=1 BORDERCOLOR=black CELLPADDING=2" +
      "CELLSPACING=0 " + "BGCOLOR=" + bak + "><TD ALIGN=center>" + 
      "<FONT COLOR=black SIZE=2>" + msg + "</FONT></TD></TABLE>";
  yyy = Yoffset;
  if (ns4) {
    skn.document.write(content);
    skn.document.close();
    skn.visibility = "visible"
  }
  if (ns6) {
    document.getElementById("dek").innerHTML = content;
    skn.display = ''
  }
  if (ie4) {
    document.all("dek").innerHTML = content;
    skn.display = ''
  }
}

function get_mouse(e) {
  var x = (ns4 || ns6) ? event.pageX : event.x + document.body.scrollLeft;
  skn.left = x + Xoffset;
  var y = (ns4 || ns6) ? event.pageY : event.y + document.body.scrollTop;
  if (document.documentElement &&  // IE6 +4.01 but no scrolling going on
     !document.documentElement.scrollTop) {
    y = event.y + document.documentElement.scrollTop;
  }
  else if (document.documentElement && // IE6 +4.01 and user has scrolled
           document.documentElement.scrollTop) { 
    y = event.y + document.documentElement.scrollTop;
  }
  else if (document.body && document.body.scrollTop) { // IE5 or DTD 3.2
    y = event.y + document.document.body.scrollTop;
  }

  skn.top = y + yyy;
}

function kill() {
  yyy = -1000;
  if (ns4) {
    skn.visibility = "hidden";
  }
  else if (ns6 || ie4) skn.display = "none"
}

我收到此错误：

“事件未定义”

在IE中可以正常工作。

Answer 1

我不会发布有关如何重写代码的代码，@ Ivo Wetzel几乎是您所需要的，但让meg给您一些建议。

1. 世界瞬息万变，计算机行业也是如此。 尽管有时速度不如我们想要的快（IE 6逐渐消失），但无需支持Netscape 4。

2. 向StatCounter之类的网站咨询，以了解正在使用的浏览器（在您所在的国家/地区）。 另请咨询YUI分级浏览器支持 。 雅虎是互联网上最大的播放器之一，其网站必须为几乎每个人使用，因此他们知道自己在说什么。

3. 找到一个好的DOM参考。 MDC几乎是您所需要的，但是拥有适用于IE怪癖的MSDN很好。 在谈论怪癖时，请不要忘记为QuirksMode兼容性表添加书签。

4. 切勿使用ie4 = document.all类的东西，因为单个功能无法识别整个浏览器。 就像在说： “嘿，你有一头金发，你一定是布拉德·皮特” 。 使用功能检测。 阅读这两篇出色的文章： 浏览器检测（以及替代方法）和功能检测：最新的浏览器脚本编写

5. 不要使用document.write因为它是同步I / O，太糟糕了。 它会阻止页面渲染，并导致不良的用户体验。 Web就是关于异步的 。

“同步编程是不礼貌的，不应在人们使用的应用程序中使用。” -道格拉斯·克罗克福德

Answer 2

噢，天哪...这一定是我多年来所见过的最糟糕的代码，那么让我们尝试清除它：

Xoffset = -60; // modify these values to ...
Yoffset = 20; // change the popup position.
var old, skn = document.getElementById("dek").style, yyy = -1000;

function popup(msg, bak) {
    var content = 
        "<TABLE  WIDTH=150 BORDER=1 BORDERCOLOR=black CELLPADDING=2" +
        "CELLSPACING=0 " + "BGCOLOR=" + bak + "><TD ALIGN=center>" + 
        "<FONT COLOR=black SIZE=2>" + msg + "</FONT></TD></TABLE>";

    yyy = Yoffset;
    document.getElementById("dek").innerHTML = content;
    skn.display = '';
}

document.onmousemove = function(e) {
    e = e || window.event;

    var x = e.pageX !== undefined ? e.pageX : e.clientX + document.body.scrollLeft;
    var y = e.pageY !== undefined ? e.pageY : e.clientY + document.body.scrollTop;
    skn.left = x + Xoffset;
    skn.top = y + yyy;
}

function kill() {
  yyy = -1000;
  skn.display = "none";
}

它仍然无法修复，但是应该可以工作……以某种方式……。好吧，除非您将其余的都发布为HTML，否则我无法对其进行测试。

拜托，我求求你...摆脱所有这些废话，并使用jQuery。

Answer 3

看来您需要将所有'event'实例更改为'e'。

Answer 4

我会测试对象/属性是否存在，而不是针对浏览器进行测试。 例如：

var x = e.pageX ? e.pageX : e.clientX + document.body.scrollLeft + document.documentElement.scrollLeft; 

我认为可能有更简单的方法可以做到这一点，例如

var x = e.pageX || e.clientX + document.body.scrollLeft + document.documentElement.scrollLeft;

但我不确定是否可以。 测试一下，看看会得到什么。 另外，有关更多详细信息，请查看： quirksmode.org/js/events_properties.html

另请注意，由于您要传递给函数的参数是“ e”，因此我将“ event”更改为“ e”。 如果您仍要使用事件，请将参数重写为：

function get_mouse(event)

尽管我不认为“事件”是JS的保留字，但许多浏览器都在使用它，因此我建议您坚持使用“ e”。

Answer 5

Firefox包括document.documentElement和document.documentElement.scrollTop以及document.body和document.body.scrollTop因此您要输入用于Firefox IE的区域。

您还应该使用类似以下内容来启动功能

function get_mouse(e) {
    e = e || window.event;

然后使用e而不是event中的所有使用场所event 。

Answer 6

添加var event = e的函数体的第一行，如果你害怕麻烦的function get_mouse(e) { var event = e;