Skeptisk till ny programmerare...

aDonis · 2012-02-13, 03:41

Tjena,

Kan någon kolla in CSSen på www.socialfrihet.se och se om allt verkligen står rätt till där? När jag kollar i style.css filen så blir jag helt förvirrad men jag kan å andra sidan ytterst lite programmering, men har aldrig sett något liknande. Tycker det ser stökigt ut och jag är orolig att programmeraren spaghettikodar sidan.

Bad honom göra om CSSen för den strulade innan och han sa att den gamla var dåligt kodad. En grej jag märkte är att han flyttade ut länken till loggan i en egen .php fil istället för att lägga den i style.css, är det normalt?

Ett utdrag från style.css:

PHP-kod:


			
html,body,div,span,applet,object,iframe,h1,h2,h3,h4,h5,h6,p,blockquote,pre,a,abbr,acronym,address,big,cite,code,del,dfn,em,font,img,ins,kbd,q,s,samp,small,strike,strong,sub,sup,tt,var,b,u,i,center,dl,dt,dd,ol,ul,li,fieldset,form,label,legend,caption{border:0;outline:0;font-size:100%;vertical-align:baseline;margin:0;padding:0}

body{color:#000;font-family:Arial, Helvetica, sans-serif;font-size:14px;background:#fff;line-height:28px; float:left; margin:0; padding:0;}

ul,ol{list-style:none}

img,fieldset{border:none}

.hidden{position:absolute;left:-9999px}

.clear{clear:both}

#header{width:946px;overflow:hidden;margin:0 auto;padding:0 0 0 19px}

.logo_area{border-bottom:1px solid #a9a9a9;width:946px;overflow:hidden;margin-bottom:25px; float:left;}



.logo_area .logos{float:left;width:440px;height:79px;padding:30px 0 42px}

.logo_area .logos a{width:440px;height:79px;display:block;}



.logo_search_area{float:left;width:328px;height:146px;border-right:1px solid #a9a9a9;padding:0 6px 5px 0; margin:0px 0px 0px 171px;}

.iconsearch_bg,.iconsearch_bg_lft,.iconsearch_bg_rgt{height:146px}

.iconsearch_bg{background:url(images/icon_search_bg.jpg) repeat-x}

.iconsearch_bg_lft{background:url(images/icon_search_bg_lft.jpg) no-repeat}

.iconsearch_bg_rgt{background:url(images/icon_search_bg_rgt.jpg) no-repeat right top}

.iconarea{width:308px;overflow:hidden;height:64px;padding:9px 0 0 20px}

.iconarea .box1{font-family:Arial, Helvetica, sans-serif;font-size:14px;color:#fff;width:79px;float:left;padding:15px 0 20px}

.iconarea .box2{width:229px;overflow:hidden;float:left}

.iconarea .box2 a{float:left;width:70px;display:block;margin-left:20px;background:url(images/icon_li_bg.jpg) no-repeat left top;padding:0 0 9px 14px}

.iconarea img{border:0;}

Vad säger ni? Behöver jag oroa mig?

abergman · 2012-02-13, 06:39

Citat:

Ursprungligen postat av aDonis

Tjena,

Kan någon kolla in CSSen på www.socialfrihet.se och se om allt verkligen står rätt till där? När jag kollar i style.css filen så blir jag helt förvirrad men jag kan å andra sidan ytterst lite programmering, men har aldrig sett något liknande. Tycker det ser stökigt ut och jag är orolig att programmeraren spaghettikodar sidan.

Bad honom göra om CSSen för den strulade innan och han sa att den gamla var dåligt kodad. En grej jag märkte är att han flyttade ut länken till loggan i en egen .php fil istället för att lägga den i style.css, är det normalt?

Ett utdrag från style.css:

PHP-kod:


			
html,body,div,span,applet,object,iframe,h1,h2,h3,h4,h5,h6,p,blockquote,pre,a,abbr,acronym,address,big,cite,code,del,dfn,em,font,img,ins,kbd,q,s,samp,small,strike,strong,sub,sup,tt,var,b,u,i,center,dl,dt,dd,ol,ul,li,fieldset,form,label,legend,caption{border:0;outline:0;font-size:100%;vertical-align:baseline;margin:0;padding:0}

body{color:#000;font-family:Arial, Helvetica, sans-serif;font-size:14px;background:#fff;line-height:28px; float:left; margin:0; padding:0;}

ul,ol{list-style:none}

img,fieldset{border:none}

.hidden{position:absolute;left:-9999px}

.clear{clear:both}

#header{width:946px;overflow:hidden;margin:0 auto;padding:0 0 0 19px}

.logo_area{border-bottom:1px solid #a9a9a9;width:946px;overflow:hidden;margin-bottom:25px; float:left;}



.logo_area .logos{float:left;width:440px;height:79px;padding:30px 0 42px}

.logo_area .logos a{width:440px;height:79px;display:block;}



.logo_search_area{float:left;width:328px;height:146px;border-right:1px solid #a9a9a9;padding:0 6px 5px 0; margin:0px 0px 0px 171px;}

.iconsearch_bg,.iconsearch_bg_lft,.iconsearch_bg_rgt{height:146px}

.iconsearch_bg{background:url(images/icon_search_bg.jpg) repeat-x}

.iconsearch_bg_lft{background:url(images/icon_search_bg_lft.jpg) no-repeat}

.iconsearch_bg_rgt{background:url(images/icon_search_bg_rgt.jpg) no-repeat right top}

.iconarea{width:308px;overflow:hidden;height:64px;padding:9px 0 0 20px}

.iconarea .box1{font-family:Arial, Helvetica, sans-serif;font-size:14px;color:#fff;width:79px;float:left;padding:15px 0 20px}

.iconarea .box2{width:229px;overflow:hidden;float:left}

.iconarea .box2 a{float:left;width:70px;display:block;margin-left:20px;background:url(images/icon_li_bg.jpg) no-repeat left top;padding:0 0 9px 14px}

.iconarea img{border:0;}

Vad säger ni? Behöver jag oroa mig?

CSS är inte min grej, men jag tycker att det ser ganska vettigt ut.

Cosma · 2012-02-13, 08:39

Jag är lite CSS perfektionist. Tycker också spontant att det ser ok ut. Jag skriver också min CSS horisontellt. Dock kanske personen borde ha kommenterat bättre. Lagt in CSS'en i sektions block osv..

Cosma · 2012-02-13, 08:41

Förresten.. Du kan alltid använda en CSS processor om du vill placera om CSS koden.

http://procssor.com/

Vebut · 2012-02-13, 12:01

De första raderna är en så kallad CSS reset för att få alla webbläsare att utgå från samma punkt och resten (.icon*) verkar helt enkelt bara komprimerat, så ser inga konstigheter i den koden inte.

tartareandesire · 2012-02-13, 19:24

Exakt vad är det du inte sett tidigare?

Inga större konstigheter med koden, vilket redan flera påpekat. Normalt sett är det dock inte CSS eller dylikt man syftar på när man talar om spaghettikod utan det begreppet brukar vara förknippat med "riktig kod", i det här sammanhanget t.ex. PHP eller .NET.

Är det ganska mycket CSS kan det vara trevligt med kommenterad blockuppdelning.

Caliber · 2012-02-13, 20:40

Du behöver inte oroa dig, css'n ser helt ok ut den är bara skriven i kompakt form. För övrigt är det inte programmering man sysslar med i css filen.

dAEk · 2012-02-13, 21:45

Ser några grejer som jag reagerar på. Inget är direkt allvarligt men jag skulle fråga varför man döpt vissa klasser som man har gjort. T.ex. tycker jag att det är ganska märkligt att döpa klasser till box1, box2, nånting_bg_lft, _bg_rgt.

En stor fördel med Css är att man separerar innehåll från design men det verkar man inte ha tänkt på om man granskar klasserna. Eller så har det varit väldigt tajt med tid, det händer ju.

Är det många .box1, box2 börjar det ringa varningsklockor men det är svårt att säga nånting mer än så utan att känna till mer info.

Oskar Lindgren · 2012-02-13, 22:01

Ser ut som att han har använt en css minifier.

jonssondesign · 2012-02-14, 02:01

Ser heller inte någonting konstigt, precis som tidigare talare har sagt.

Jag skriver också min css horisontell, på så sätt slipper jag scrolla i tusen år för att hitta en sak.

Dock kan jag hålla med om att sektionsindelning vore bra, speciellt om det är många klasser och id'ns.

Men utöver det behöver du inte oroa dig!

Aktiva användare som för närvarande tittar på det här ämnet: 1 (0 medlemmar och 1 gäster)

Menu

Skeptisk till ny programmerare...