Thema: Bitte Design und Code bewerten! [EDITOR]

Zitat von Palla92

ich befürchte, dass der semantisch (immer noch) nicht korrekt ist

Du benutzt <p>, um zwei Zeilenumbrüche einzufügen. <p> zeichnet aber einen Textabsatz aus. Dazu gehört auch immer ein schließendes Tag.

Code:

<p>Herzlich willkommen auf <span>DP Fotografie.com</span>, dem Internet Projekt von Diana Palavandishvili.</p>

Ich hab aus dem <b> (das weil es physische Inhaltsauszeichnung ist und keine Semantik beschreibt, nichts im HTML-Code zu suchen hat) mal in ein <span> geändert, weil ich nicht weiß, welche Bedeutung Du mit der Fettschrift vermitteln willst. Wenn es nur fett sein soll ohne Bedeutung, dann bleibt <span> und Du nutzt CSS zum fett machen. Wenn es wichtig sein soll, dann nutzt du <em>, wenn es sehr wichtig sein soll <strong>.
<br> ist physische Inhaltsauszeichnung und hat daher hier auch nichts verloren.
Wenn Du unbedingt willst, dass Dein Textabsatz nur eine bestimmte Breite hat, dann gib für das <p> eine fest Breite (width) an.

div#foot wäre als <p> besser ausgezeichnet, es handelt sich immerhin um Text und nicht um eine Gruppe ("Gruppe" ist die Bedeutung von <div>).

div#grafik sollte wohl auch keine Gruppe sein (enthält ja auch nur ein Element), sondern eine Überschrift. Wahrscheinlich ersten Grades, also ersetze das <div> durch ein <h1>.

Die Adresse im Impressum kannst Du auch als <address> auszeichnen, da sind die <br> erlaubt, weil sie eine logische Einteilung bedeuten. Aber das <b> muss natürlich auf allen Seiten weg.

IGuG ist das Ergebnis ganz gut, nur die Bedeutung von <p> hat sich Dir bisher wohl nicht erschlossen. Ich hoffe, das ist jetzt klar.

Wenn nicht, fragen!

Zitat von vitus37

Das dachte ich mir auch, aber der Validator hat keine Fehlermeldung gegeben. Weshalb?

Wahrscheinlich, weil die HTML-Syntax kompliziert ist.

Zitat von Palla92

nur bei dem Footer will ich eigentlich nicht aufs div verzichten, weil ich den Absatz (den p nun mal erzeugt) nicht rauskriege

Das ist aber sinnfrei. Das wäre so, wie wenn Du sagst, dass Du zum Briefkasten um die Ecke lieber mit dem Auto fahren willst, weil Du nicht weißt, wie Du Deine Schuhe putzen kannst.
Und: <p> erzeugt nicht "nunmal" einen Absatz. Es gibt in CSS keine Vorschriften, wie ein Browser ein Element darzustellen hat, also mit welchen Default-Eigenschaften es belegt ist. Es gibt zwar eine Empfehlung, und in diesem Falle halten sich bisher auch alle dran. Aber wissen kann man das nicht.
Wenn Du den Absatz nicht willst, dass musst Du den Absatz entfernen. Wenn Du stattdessen ein anderen HTML-Tag nimmst, änderst Du ja die Bedeutung Deines Inhalts (hier zu etwas völlig sinnfreiem).

Abstände setzt eman über die CSS-Eigenschaften "margin" (Außenabstand) und "padding" (Innenabstand).