Design und Code bewerten (EDITOR)

Hallo Community,

für einen Betrieb, in dem ich momentan ein Praktikum absolviere erstelle ich eine Webseite.

Das Design habe ich aus einer Vorgabe erstellt, wird also auch nicht weiter geändert und der Inhalt ist auch noch nicht komplett.

Ansonsten würde ich mich freuen, wenn ihr Fehler im Quellcode oder Anzeigefehler findet, oder sonstige Ideen/Vorschläge oder Kritiken habt.

Hier ist die zu bewertende Seite
(Momentan testweise auf meinem Webspace online)

Freue mich jetzt schon auf die Antworten,

Lg :wink:

Hi!

Also von Desgin her ansprechend aber ich find es doch ein wenig störend wenn man z.B. auf der Seite Kontakt so scrollen muss, scroll frei würde es mir besser gefallen :wink:

Zum Code: Grauenhaft so weit ich es beurteilen kann :wink:
Tabellenlayout! Das ist schon seit einigen Jahren veraltet nutzte CSS dann wird der Code um einiges übersichtlicher und du gehst der Gefahr von groben Anzeigefehlern weites gehend aus dem Weg.

Ansonsten gefällt mir die Seite!

T02

Ja das mit dem Scrollen - Werde ich wohl noch ändern, außer beim Impressum, da ist der Text doch ein bisschen zu lang.

Zum Code: Grauenhaft so weit ich es beurteilen kann :wink:
Tabellenlayout! Das ist schon seit einigen Jahren veraltet nutzte CSS dann wird der Code um einiges übersichtlicher und du gehst der Gefahr von groben Anzeigefehlern weites gehend aus dem Weg.

Also ich weiß garnich was an ein paar Tabellen so schlimm ist?

Meinst du ich soll komplett die Tabellen raus lassen und nur mit CSS und DIV’s arbeiten?!

Lg

Wieso Div’s? Man sollte mit Div’s sehr sparsam umgehen.

MfG xXxPeterPanxXx

Hm, dann würde es mich freuen, wenn mir jmd. andere Möglichkeiten als Tabellen und Div’s zeigen könnte.

Lg

Ok, eine andeme möglichkeit.

Benutze „div“ um die Seite einzuteilen. Allerdings sehr sparsam.
Nutze „div“ nur, wenn in dem div minderstens 2 verschiedene andere tags vorkommen wie beispielsweise „p“ und „ul“.

Benutze für alles den richtigen HTML Tag. Für Listen nimmst du „ul“, für Absätze „p“ und für Überschriften h<1-6> (bsp. h1).

HTML zeichnet den Text sinnvoll aus. Ein Layout wird nicht aus einer Tabelle gemacht. Tabellen sind nur dafür da tabellarische Inhalte zu zeigen.

Moment mal.

Das Layout der Seite ist doch in „sparsamer Anzahl“ an Div’s eingeteilt. (Header, Content, Footer) ← Das ist doch wohl wirklich nicht viel…

Und die HTML-Tags kenne ich, danke. :expressionless:

Lg

layout finde ich ansprechend, der scrollbereich beim kontakt ist ein argument, mach den content bereich so gross, dass man dort nicht scrollen muss und dann ist es gut.

die navigation finde ich gelungen und überlegt, mal was anderes und wirklich gut gelungen.

layout absolut ok. valide und sparsam. ich finde gesamturteil gut angesagt.
die tabelle stört mich garnicht, man könnte z.b. deine navi sogar als tabellarisch ansehen.

das prinzipielle no tabel kann man sehen, wie man will, päpstlicher sein wollen als der papst und dann oftmals nicht mal validen code zustande bekommen…

Er spricht mir aus der Seele :razz: …