Du hast recht, es sieht besser aus, aber das wars auch schon.
Der Code hat sich nicht wirklich verbessert. Du hast die Frames raus genommen, die Semantik aber nicht wirklich verbessert.
Werde dir bewusst für was HTML da ist. Mit HTML sagst du WAS ein Textabsatz ist, WAS eine Überschrift ist, WAS ein Bild ist, WAS ein Liste ist.
Und in CSS gibst du dann an WIE all das dargestellt werden soll.
Du nutzt DIVs um dein Design damit zu machen, da es aber HTML Elemente sind, haben sie nicht das geringste mit dem Aussehen zu tun. Dieses Element ist dazu da mehrere andere Elemente zu gruppieren.
Ist also nicht korrekt, es beinhaltet kein einziges Element.
Bei
HTML:
<p><img src="wappen.jpg" alt=""></p>
sind die p-Tags unnötig.
HTML:
<font color=white><h1>Herzlich Willkommen auf der FC Energie - Fanseite</h1></font>
ist ebenfalls falsch, da du hier HTML dazu benutzt die Schrift zu formatieren.
HTML:
<div class="Copyrights">© by warstormzone.de.tl 2010</div>
In diesem DIV ist auch kein Element, nur ein Text, hier solltest du das DIV durch zB P ersetzen.
Da eine Navigation eine Liste aus mehreren Links ist, wird diese auch als Liste ausgezeichnet. Also mit <ul> und <li>.
Wenn du all deinen Inhalt korrekt ausgezeichnet hast, kannst du damit beginnen jedes einzelne Element zu formatieren, und das komplett mit CSS.
Es sind auch noch eine Menge Syntax-Fehler vorhanden:
[Invalid] Markup Validation of http://www.angelfire.com/warst0rm/index.html - W3C Markup Validator
Ist dieses ganze JavaScript Zeugs bei dieser Seite eigentlich nötig? Ich glaube nicht. Diese MouseOver Effekte bei der Navi bekommst du besser mit CSS hin. Kenne mich mit JS nicht so aus, weiss deshalb auch nicht wofür die anderen Scripts alle sind, bin mir aber sicher du brauchst die nicht.
MfG