Code bewerten/korrigieren!
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
... 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;
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
... 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;
user-300
15.06.2006 08:14
Schaut gut aus. Browsertests kannst du auf http://browsershots.org machen
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;
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;
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
Auf jeden fall ist es viel einfacher zu lesen.
user-289
15.06.2006 12:42
für mich nich.. Hab doch nachgezählt
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
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
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
Wegen einem Zeichen muss man sich hier aber definitiv nicht streiten. Wie ich schon sagte: margin-xy ist viel einfacher zu lesen.