Hi Sebastian!
Jeder fängt mal an und dafür finde ich es gar nicht mal schlecht. Habe mir direkt mal den Quellcode angeschaut und er ist okay. Es gibt ein paar Dinge, die du einfacher machen könntest.
Beispielsweise würde ich die Navi oben in einen extra div-Container legen und lediglich diesem Container die CSS-Klasse zuweisen. Denn 100 Mal die gleiche Klasse zuweisen...da könntest du die Attribute auch fast direkt selbst dann dort reinschreiben.
Also dann in CSS:
- Code: Alles auswählen
.menu ul
.menu li
Den Teil unter dem Menü würde ich dann ebenfalls mit div-Layern realisieren, wobei die Methode über die Liste natürlich auch elegant ist, wenn auch nicht unbedingt notwendig.
Was mir auch noch aufgefallen ist:
Falscher Doctype, denn teilweise arbeitest du schon nach den neuen "Richtlinien" (<br />), teilweise aber auch nach den Alten (<br> und <img ...>). Da würde ich mich dann komplett an die neuen Richtlinien der W3C halten.
Grafisch: Über sowas lässt sich ja bekanntlich streiten. Mein Fall ist es nicht, da die Farben teilweise sehr "blenden" und (wie beispielsweise oben rechts das grün) kaum lesbar sind. Ob die Kombi aus grün/blau da der richtige Weg ist weiss ich auch nicht...
Zudem hängt die Seite irgendwie im leeren Raum. Nichts beschränkt sie und keiner weiss wo sie aufhört. Was ich dann noch hinzufügen würde wäre im Content-Bereich einen Padding-Wert, damit die Schrift nicht ganz am Rand hängt.
Lieben Gruß,
Manuel
![Extrem frech ;-]](./images/smilies/icon_impudently.gif)