Kleine Anmerkung im Voraus: Ich bin mir nicht so ganz sicher, ob es sinnvoll ist, eine fertige Seite noch mal getrennt im Showcase vorzustellen. Die Erfahrung zeigt, dass User hier häufig die Wünsche zum Bewertungsrahmen nicht lesen oder sie ignorieren und in beiden Kategorien zu beidem kommentieren. Im Showcase sind Kommentare konkret zum vorliegenden Code nicht angebracht, hier im Websitecheck hingegen schwer zu vermeiden. Wenn du aber eine strikte Einhaltung wünschst, kannst du gerne darum bitten; die Unterstützung der Mods hättest du. :)
Zum Code der Seite:
Allgemein sehr schön schlank, strukturiert, valide und hauptsächlich semantisch nachvollziehbar. Sieht man zwar immer häufiger, aber immer noch zu häufig das Gegenteil. Deswegen erst mal ein Lob dafür.
Jetzt noch ein bisschen Kritik. Das wird nach mehr aussehen als es ist, weil ich eben gerne genau erläutere, was ich meine und warum:
Wie Icy schon anmerkte, läufst du schnell Gefahr, monströse div-Konstruktionen zu bauen, wo sie nicht unbedingt notwendig sind. Hier den Ausschnitt
Code:
<div id="headernavigation">
<ul id="header_navigation">
<li>[...]</li>
<li>[...]</li>
</ul>
</div>
kann ich z.B. kaum nach vollziehen.
div ist ein Element, welches dazu dient, Elemente zu gruppieren, die zusammen gehören. Diese Aufgabe übernimmt hier aber schon
ul. Eigentlich bläht das
div hier deinen Code nur auf. Es scheint auch keine konkrete gestalterische Aufgabe zu übernehmen, die nicht ebenso gut
ul bekommen könnte.
Ich bin jetzt nicht jeden einzelnen Fall durchgegangen, wo viele
divs geschachtelt waren, aber du kannst bestimmt noch ein paar einsparen. ;)
@mermshaus: Da gebe ich dir prinzipiell Recht, aber
<div><ul><li></li></ul></div> ist so ein Klassiker, bei dem man schon drei Sorten variabler grafischer Elemente über-/aneinander haben müsste, um ihn zu rechtfertigen. Vor allem, weil es hier ja auch noch die Hauptnavigation direkt unter dem Seitenkopf ist.
Etwas irritierend finde ich die Tatsache, dass der Header, welcher optisch ganz oben auf deiner Seite steht, im Code ziemlich weit unten kommt. Wo jetzt "Kontakt" und "Impressum" stehen, ist ja eigentlich egal (wahrscheinlich ist im Dokument unten dafür sogar sinnvoller), aber du bindest deine oberste Überschrift quasi zweimal ein: Zum Sehen auf der Seite (die Grafik) und für den, der in den Code guckt. Da erschließt sich mir der Sinn so gar nicht.
Ich rieche SEO, aber du kannst ja auch einfach eine Image-Replacement-Methode nutzen. Den Text der
h1-Überschrift machst du ja anscheinend so oder so per CSS unsichtbar. Es würde wieder ein bisschen Code sparen (neben dem
img auch noch ein
div).
Ansonsten sorgt noch die Art und Weise, wie du deine Seite zentrierst, dafür, dass sie sich beim Verkleinern des Fensters über den linken Rand hinausschiebt, während der Scrollbalken aber ganz links "am Anschlag" bleibt. Das geht natürlich gar nicht. Netbooks und internetfähige mobile Endgeräte (sprich: Smartphones, PDAs) gibts schon eine Weile, inzwischen kann man sie aufgrund ihrer Verbreitung kaum noch ignorieren.
Deine Seite verschwindet schon z.B. bei 1024x768 (nach wie vor zu berücksichtigen!) und angezeigter Vista-Sidebar nach links. Jetzt noch links irgendeine Browser-Leiste aufklappen und man kann gar nichts mehr damit anfangen. Möglicherweise reicht auch schon die Sidebar, ich kanns jetzt nicht ausprobieren.
Das waren echt nur drei Punkte, sieht bloß so viel aus. :mrgreen:
