Optimere Javascript til de mest gængse browsere

Det lyder nærmere som en arbejdsopgave !-)

tjaaa, man kan jo starte med at finde nogle links frem :-)

Gode ideer til at få følgende side til at fungere i FireFox modtages gerne - og belønnes med 10-15 points pr. ide:
http://www.mountfield.dk/realtime/page.php

Mvh, Steven

først og fremmest bør du droppe den browser-sniffing du bruger (isMinNS4 osv.) Istedet lave en feature-test, dvs. teste om en feature er understøttet af browseren når den skal bruges.

Fx når du skal have fat i et element, så kunne du skrive:
if (document.getElementById) {
    elm = document.getElementById("elementID");
}
else if (document.all) {
    elm = document.all["elementID"];
}

Typisk bruger jeg en lille funktion til den slags:
function getElm(elmId) {
    if (document.getElementById) {
      return document.getElementById(elmId);
    }
    else if (document.all) {
      return document.all[elmId];
    }
    else return null;
}

Så kan du kalde den sådan her: elm = getElm("elementId");
Som du kan se har jeg ikke tager document.layers med - den kan du tilføje hvis du lyster, men jeg er efterhånden holdt op med at understøtte de gamle netscape browsere på den sigt. document.layers var så fuld af fejl og mangler så moderne scripts vil sikkert ikke virke i den selvom man prøvede at bruge document.layers.

Hej Skovenborg,

Tak - jeg har benyttet din funktion, og det virker :-)
Giv et svar, og så får du 15 points!

Har du en ide til hvorfor MoveLayerTo functionen ikke virker i FireFox?

function moveLayerTo(div, x, y)
{
  if (isMinNS4)
  {
    div.moveTo(x,y);
  }
  if (isMinIE4)
  {
    div.style.left = x;
    div.style.top  = y;
  }
}

Mvh, Steven

fordi du mangler enheder, som er påkrævet i Mozilla (IE har som standard pixel, hvorfor det virker der). Dit script skal derfor se sådan ud:
function moveLayerTo(div, x, y)
{
  if (isMinNS4)
  {
    div.moveTo(x,y);
  }
  if (isMinIE4)
  {
    div.style.left = x+"px";
    div.style.top  = y+"px";
  }
}

Fordi længder i css ifølge standarden _skal_ overses, hvis de ikke har enheder på, så:

function moveLayerTo(div, x, y)
{
  div.style.left = x + 'px';
  div.style.top  = y + 'px';
}

-- og drop understøttelse af gamle browsere, medmindre du er pisket !-)

-- og kaldet skal så have objektet med:

moveLayerTo(document.getElementById('minDiv'), 15, 15);

eller moveLayerTo(getElm('minDiv'), 15, 15); hvis du stadig har den funktion liggende.

Hej, tak skal I have - nu virker det :-)
Roenving, vil du ikke lige smide et svar også, så giver jeg jer følnende points:
Skovenborg: 15+10
Roenving: 10

Takker for hjælpen!

...hvis I har flere gode ideer må I gerne sige til :-)

Oki '-)

svar for at få point til at gå op...

Tak for point ;~}