Hi,
zunächst mal fällt negativ auf, daß Du in PHP exakt denselben Fehler machst, der hier ständig bei CSS moniert wird. Du bezeichnest z.B. menu_ul = 'ul', also implizierst Du, daß (natürlich meist sinnvoll) eine ul verwendet wird. Besser wäre hier freilich, sich auf die Funktion zu beschränken, also z.B. menu_container.
Bei Deiner Seite fällt optisch zunächst der wirklich ungenügende Kontrast auf, aber auch der durch überlange Code-Zeilen erzeugte unschöne horizontale Scrollbalken.
Beim Menü-Quelltext auf Deiner Seite wundere ich mich über das <div id="menu">. Ich sehe hierzu keine Notwendigkeit.
Ach ja: nur eine für Dich vielleicht unwichtige Nebensache, aber ich finde Quelltexteinrückungen bei verschachtelten Elementen äußerst nützlich.
freundliche Grüße
Ingo