Struppi: Drop Down Menü

Beitrag lesen

navtop = function() {

if (document.all && document.getElementById) {

hier ist der Test auf document.all überflüssig.

navRoot = document.getElementById("wrapNav");

for (i = 0; i < navRoot.childNodes.length; i++) {

globale Schleifenvariabeln, das schlimmste was du machen kannst.
Überhaupt solltest du die Variabeln immer mit var lokal machen, dann können sie sich nicht in die Quere kommen.

node = navRoot.childNodes[i];

if (node.nodeName == "LI") {

Hier würde ich mal testen ob LI überhaupt Grossgeschrieben wird oder zur Sicherheit gleich:
if (node.nodeName.toUpperCase() == "LI") {

Wobei, warum benutzt du nicht getElementsByTagName()?

node.onmouseover = function() {

this.className+= "over";
     }

Mehrere Klassen müssen mit einem Leerzeichen getrennt werden.

<div id="wrap">
<div id="navTop">

<!--1. navigation top-->
  <div id="posDiv">

Divitis?
(aber tpisch für vermutlich Wordpress Themes)

Struppi.

--
Javascript ist toll (Perl auch!)