http://validator.w3.org/check?verbose=1&uri=http%3A%2F%2Ffotogen.clubgreen.dk%2F<div class="toppen">Fotogen</div> -- det ligner unægteligt en overskrift, hvorfor ikke bruge et hx-element? Samme gør sig gældende flere andre steder.
<body><br /> -- Væk med det br-elementet og anvend CSS i stedet. Forresten bør du vist også smide noget af den CSS du har i style-elementet over i den externe fil. Og i det mindste huske at angive en generisk font-klasse sammen med den du har valgt.
Og footer'en ville jeg helt klart anvende et p-element til.
Rent visuelt er siden dog pæn! Men den har også lidt problemer i IE7B2; du bør nok redesigne den, så den først og fremmest ser pæn ud i en forholdsvis standards compliant browser (dvs. ikke IE), og derefter tilføje fixes til de andre (formentlig IE først og fremmest). Det er næsten altid den nemmeste fremgangsmåde.