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!)