teststudio nur Code Bewerten

[Shrax]

Guten Tag.
Ich hab mit meiner Website immer wieder Layoutprobleme deswegen lerne ich erstmal weiter css.

Ich dachte mir am besten kann ich lernen wenn ich versuche einen sehr übersichtlichen Code zu machen.

Bin jetzt nach langer Arbeitszeit fertig. Da ich noch am css lernen bin hab ich mich noch schwer getan.

Achtet nicht auf die farben das Layouts den diese habe ich nicht beachtet.
Mir ging es erstmal darum einen guten Code zu schreiben. Darum würde ich mich Freuen wenn ihr den CODE bewerten würdet und evt. Verbesserungsvorschläge bringt.

Den css lage ich normalerweiße in eine Extra Datei aus.
Daniel K Internet Studio - Startseite

 

[Sn0opy]

Statt <style> musst du schreiben: <style type="text/css">

Ansonsten sind es für mich persnönlich zu viele Tabs.Der Head Tag fängt normalerweise ganz links an und nicht direkt mit einem Tab davor. Übersichtlicher wäre es außerdem, wenn der CSS Code in einer Datei verschwinden würde.

Ansosnten würde ich den einen oder anderen Kommentar einfach nochmal rauslassen. So viele machen die Übersicht dann doch wieder kaputt

 

[prm]

Darüber hinaus folgende Anregungen:

Du setzt die Abstände aller Überschriften auf "Null". Kann man so machen.
Sinnvoller, auch um unterschiedliche Browserdarstellungen zu vermeiden, wäre es jedoch, wenn du mit dem Universalselktor die Abstände sämtlicher Elemente auf "Null" setzt und sie anschließend wieder einheitlich hinterlegst. Auf diese WQeise kannst du bei "h" entweder den Null-margin entfernen, oder aber einen individuellen Wert hinterlegen.
Wie das funktioniert steht hier erklärt: Unterschiedliche Abstände in verschiedenen Browsern

Für html und body definierst du "text-align: center;" um ihn beim #container auf left zu stellen.
Lass besser beide Angaben komplett weg. left ist die Standardeinstellung. Definiere center nur bei den Elementen, wo es auch verwendet wird.

Die Schriftart ist in fast allen Elementen hinterlegt. Einmal im body mit

font-family: arial, hevetica, sans serif;

reicht vollkommen aus.
Den Elementen dann nur das zuweisen, was von der Generalangabe abweicht.

Kurzschreibweisen verwenden:
Statt bisher

padding-top: 6px;
padding-right: 4px;
padding-bottom: 4px;
padding-left: 6px;

besser:

padding: 6px 4px 4px 6px;

Erklärung siehe hier, Ziffer 4: Effektiver CSS-Code

Auch die Rahmenangaben kannst du mit Kurzcode einfacher darstellen.

Weniger Div's verwenden:
Bisher:

 <!-- Start Header -->
                   <div id="header" title="istudio"><h1>Test</h1></div>
                             <!-- End Header -->

Besser:

 <!-- Start Header -->
                  <h1>Test</h1>
                             <!-- End Header -->

Sämtliche Angaben von #header kannst du genauso gut auch h1 zuweisen.
Position: relative; ist nicht erforderlich.

Das sind mal die Dinge, die mir so spontan aufgefallen sind.

Je mehr du den CSS- und HTML-Code abspecken und vereinfachen kannst, je übersichtlicher und damit weniger fehleranfällig wird er.
Ansonsten sieht's für einen Anfänger aber schon recht ordentlich aus.

 

[Shrax]

Supi genau so eine Kritik wollte ich vielen Dank, ich bessere gleich alles aus undbei meinen nächten design mach ichs gleich so :)

 

[Tobias]

Ja am Design solltest du auch unbedingt was tun
Aber wünsche dir trotzdem viel Spaß und Erfolg

Ps.
Bei deinem Username fehlt ein "e".
Es wird Hurricane geschrieben. ;)

 

[Maxi]

Ps.
Bei deinem Username fehlt ein "e".
Es wird Hurricane geschrieben. ;)

Schreibt man normal auch nicht mit 3 "r" ;-)