Moin,
als erstes missfällt mir die schweinchenrosa Hintergrundfarbe. Die passt nicht zu Fröschen.
Dann ist die Seite völlig unnützerweise breiter als mein Browserfenster, was mir horizontale Scrollbalken beschwert, was mich schon wieder tierisch (wie passend ;) nervt.
Ich persönlich mag kleine Fonts, aber ich weiß, dass es etliche Leute gibt, denen diese Schriftgröße wahrscheinlich schon zu klein ist.
Ich würde Dir vorschlagen, Deinen Header wegzuschmeissen, stattdessen das nur noch den Frosch ins Logo aufzunehmen, und die Überschrift als solche in die Seite einbinden (<h1>), also als Text. Dann gibt es keinen Grund mehr, warum die Site eine fixe Breite hat und kann sich wunderbar den Browserfenstern Deiner Besucher anpassen. Keine Horizontalen Scrollbalken mehr.
Dann würde ich die Hintergrundfarbe auf weiß setzen, das erspart Dir dann auch in Deiner Überschrift einiges.
Zur Technik:
Schön finde ich, dass Du keine Tabellen zu Layoutzwecken missbrauchst. Jedoch solltest Du Dein Wissen um CSS noch erweitern. Du solltest all Deine Layout-Angaben in ein externes Stylesheet auslagern, was zu schnelleren Ladezeiten und übersichtlicherem Code führt. "Trennung von Inhalt und Layout" ist, was Dir hier noch fehlt.
Und Du solltest semantisches Markup gebrauchen. Nutze die richtigen HTML-Tags an der richtigen Stelle!
Warum z.B. dieses:
<div style="height:150px;">
<img src="../styles/pics/logo.jpg" alt="Das FROSCH-Prinzip">
</div>
Das ginge doch so viel besser (semantisch richtiger!):
<h1 style="height:150px;"><img src="..." alt="Das FROSCH-Prinzip"></h1>
Das braucht kein div (Vorsicht vor der div-Soup) und die Überschriftsgrafik wird auch als das ausgezeichnet, was sie ist: Eine Überschrift!
Auch semantisch sinnvoller ist es, ein Menü als das anzugeben, was es ist, nämlich eine Liste (<ul>). Damit kannst Du Dir die ganzen <br>s sparen, die nach Ansicht mancher eigentlich zum Layout gehören und damit nichts in der HTML-Datei zu suchen haben ("Trennung von Inhalt und Layout").
<span class="green"> ist z.B. völlig aussagelos. Frage Dich, warum der Text "FROSCH-Prinzip" hier grün ist, und suche nach einem passenden Tag. Wenn es Dir darum geht, den hervorzuheben oder zu betonen, dann gibt es <em> oder <strong>, für einen feststehenden Begriff gibt es sicher auch Tags *wühl*kram*such* hier: http://de.selfhtml.org/html/text/logisch.htm#elemente
Also von der technischen Seite gefällts mir, auch wenn Du Dein Wissen noch verfeinern kannst. Optisch stört mich am meisten die rosa Hintergrundfarbe und der Scrollbalken, der nicht nötig ist.
Ansonsten ist es optisch vielleicht ein bisschen langweilig. Beim Frosch-Workshop könnten doch vielleicht Bilder von dem Workshop einfließen?
Gruß,
-Efchen