Struppi: dynamisches menue

Beitrag lesen

Sorry, ich hab deinen Quelltext nur kurz überflogen. Ich finde diese Menüs schrecklich überflüssig. Aber trotzdem kann ich mir zum Code eineige Bemerkungen nicht verkneifen.

body,p { font-family:"Verdana"; font-size:8pt; color:#ffffff; }
tr,td { font-family:"Verdana"; font-size:8pt; }

'pt' ist kein Maßeinheit für den Monitor sondern für die Druckausgabe, wo ein pt genau ein Punkt ist auf dem Moitor heissen diese Pixel.

Deine Seite dürfte auf den meisten Rechnern nicht mehr lesbar sein mit diesne Angaben.

<SCRIPT LANGUAGE="JavaScript">

hier fehlt das type Attribut type="text/javascript"

if (navigator.appName=='Netscape' && document.layers!=null)
{ pageLayers = document.layers; }
else if(document.all)
{ pageLayers = document.all; }

Du möchtest dieses Skript, dass lediglich auf 4'er Browser teste nicht mehr wirklich verwenden oder?

Damit wird deine Seite in den meisten modernen Browsern unbrauchbar (ein Grund mehr solche Menüs nicht zu verwenden, es gibt aber einige mehr).

var bild = (document.all) ? eval("document.all.bild"+nr) : eval("document.ID"+nr+".document.bild"+nr);

eval ist evil!

und hier absolut überflüssig. Sorgt nur für Ärger.

if(flag=="hi")

wenn flag wirklich ein flag ist warum nutzt du nicht true/false oder 0/1?

function MM_reloadPage(init) {  //reloads the window if Nav4 resized

Eine ganz üble Funktion. Sie soll suggerieren wir kümmern uns um die armen NC 4. benutzer, doch die sollten mittlerweile Wissen, welche Mängel ihr Browser hat

<body bgcolor="#FFFFFF" link="#999999" vlink="#888888" alink="#666666">

Das solltest du alles in den CSS Block packen, dafür ist CSS da.

<div id="Layer1" style="position:absolute; left:417px; top:35px; width:63px; height:23px; z-index:8"><a href="#" onMouseOver="change_button(1,'hi')" onMouseOut="change_button(1,'low')"><font color="#000000"><img border=0  name="bild1" src="bild01.gif" width="1" height="1" align="middle"><font size="1" face="Verdana, Arial, Helvetica, sans-serif">
  Menue</font></font></a></div>

font ist ein veraltetes Tag und soll nicht mehr verwendet werden -> nimm CSS dafür.

Das Skript braucht ein Update.

Struppi.