Webstatt.org - Community seit 2006 - 2012 (2024?)

Code bewerten/korrigieren!

Avatar user-289
15.06.2006 01:18

Hallo,
hab hier mal den größten Teil meiner neuen Seite fertig gecodet.
Ich würde mich freuen, wenn ihr mal in den Code (insbesondere CSS) reinschaut und mir ggf. Verbesserungsvorschläge macht. Das ganze ist XHTML 1.1 Strict Valid.

Hier der Link: http://www.t-sdesign.de/preview/v2/index.html

Achja: Ein Browsertest hätte ich auch gerne frech... Weiss jemand wie ich die Seite im Opera an "die Decke" bekomme? Es ist immer etwas margin vorhanden...

Edit: Ahhh hab schon! Mit padding: 0; zwinkern

Interesse? meine Referenzen. hier!!!
Avatar user-300
15.06.2006 08:14

Schaut gut aus. Browsertests kannst du auf http://browsershots.org machen zwinkern

Ich würde das id Attribut nur für grössere Teile benutzen, z.B. für die Navigation als ganzes. Die einzelnen Navigationselemente kannst du ruhig mit class definieren. (Eine ID muss einmalig sein in einem Dokument und dient gleichzeitig als Anker.)

Sachen wie "margin: 0 0 0 8px;" kannst du einfacher schreiben: margin-left: 8px;

Alle Eigenschaften des Hintergrundes kannst du zusammenfassen:
background-color: #828282 url(../img/backgrounds/bg_01.jpg) repeat-x;

Avatar user-289
15.06.2006 11:25

Danke, das mit dem Hintergrund wusst ich noch nicht. Spart auf jeden fall etwas Länge im Code...

Das mit dem margin... Mh naja, also meine Variante ist kürzer, aber im Prinzip ist das ja egal...

Das mit dem id-Attribut ist schon so ok denke ich. Wenn man eine Deklaration einem einzigen Tag zuteien will sollte man auch immer id nehmen.

Interesse? meine Referenzen. hier!!!
Avatar user-300
15.06.2006 11:36

Original von user-289
Das mit dem margin... Mh naja, also meine Variante ist kürzer, aber im Prinzip ist das ja egal...


Zähl doch nach frech
Auf jeden fall ist es viel einfacher zu lesen.

Avatar user-289
15.06.2006 12:42

für mich nich.. Hab doch nachgezählt frech

margin-right: 1px; <- 18 Buchstaben
margin: 0 1px 0 0; <- 17 Buchstaben

margin-left: 1px; <- 17 Buchstaben
margin: 0 0 0 1px; <- 17 Buchstaben

margin-bottom: 1px; <- 19 Buchstaben
margin: 0 0 1px 0; <- 17 Buchstaben

margin-top: 1px; <- 16 Buchstaben
margin: 1px 0 0 0; <- 17 Buchstaben

Interesse? meine Referenzen. hier!!!
Avatar user-300
15.06.2006 18:13

Original von user-289
margin-right: 1px; <- 18 Buchstaben
margin: 0 1px 0 0; <- 17 Buchstaben // 18

margin-left: 1px; <- 17 Buchstaben
margin: 0 0 0 1px; <- 17 Buchstaben // 18

margin-bottom: 1px; <- 19 Buchstaben
margin: 0 0 1px 0; <- 17 Buchstaben // 18

margin-top: 1px; <- 16 Buchstaben
margin: 1px 0 0 0; <- 17 Buchstaben // 18


Mal zählst du mit Semikolon, mal ohne. Das ist unfair frech
Wegen einem Zeichen muss man sich hier aber definitiv nicht streiten. Wie ich schon sagte: margin-xy ist viel einfacher zu lesen.

Avatar user-289
15.06.2006 18:37

Huups.. naja egal frech

Interesse? meine Referenzen. hier!!!