Hallo Dominik,
die Seite ist schon okay, der Code ist auch okay. Aber hier und da gibt es schon noch ein wenig zu verbessern. Der Tennisball gefällt mir auch nicht, auch die Richtung in die die Flammen gehen nicht. Wieso brennt er nach oben? Normalerweise fliegt ja ein Ball, also vielleicht eher wie ein Komet gestalten? Die Navigation passt - gestaltungstechnisch - nicht zum Header und Content.
Es fehlt vllt. etwas im Impressum. Ich kenne mich mit der Rechtslage in Österreich allerdings nicht aus. In Deutschland müssten Name, Anschrift und Telefon noch rein.
Zur Codebasis: Wieso umschließt du die Navigation mit einem div? Divs benutzt man zum Gruppieren, eine Gruppe besteht aber mindestens aus zwei Elementen! Du hast aber nur ein ul drin. Vollkommen unnötig dieses mit einem div zu umschließen. Du kannst schließlich auch ul per CSS gestalten.
Ein div ohne Inhalt. Das ist im Prinzip noch schlimmer als oben. ;) Ersetze das doch durch
HTML:
<h1><span>Sandro Steiner</span></h1>
Denn eigentlich ist "Sandro Steiner" deine erste Überschrift. Wegen Barrierefreiheit gehört dort auch entweder der Text, den das Hintergrundbild enthält, da rein (mit den spans kannst du den Text ja per CSS wieder ausblenden, Stichwort
visibility:hidden) oder noch besser direkt den Text als Bild einbinden und per alt-tag beschreiben.
Kleiner Fehler noch im Footer: Bei © fehlt ein Semikolon zum Abschluss, also ©
Besten Gruß
Körnerbrötchen