Medlem
|
|
Reg.datum: Jan 2006
Inlägg: 282
|
|
Medlem
Reg.datum: Jan 2006
Inlägg: 282
|
Det enda jag har att klaga på skulle vara att sidan ligger lite långt till vänster. Centrerat i mitten tycker jag personligen skulle vara bättre på den här sidan, det kan fås såhär i bigdiv:
font-size:15px;
margin:0 auto;
position:relative;
width:800px;
Här kommer lite kort om css-koden bara för skojs skull. Ingenting som direkt stör, men vi alla lär så länge vi lever...
Just nu ser strukturen ut såhär:
- BigDiv
-- BoxOne
--- black/img
---- menucontainer
----- ul
Det finns ingen direkt poäng såvitt jag kan se att ha BigDiv OCH BoxOne, du kan slå ihop dem. Sedan kan man img-taggen och använda bakgrund ist. på black. Menucontainer behövs inte heller eftersom ul kan fungera som en container, så då får man:
- BigDiv
-- Black
--- UL
Vilka logiskt kan döpas så att man enkelt ser vad det är:
- Mastercontainer
-- Header
--- UL (id = menu)
Sedan kommer vi ner till själva innehållet. Tables, INTE okej. Helt onödigt. Istället för att ha en td och sätta bredden på den kan du skapa en semantiskt korrekt div och sätta bredd på den istället, sedan lägger du till float left. Sedan gör du en till div under den och sätter in höjd och bredd och skriver float right. På så sätt får du två divs istället för tre tds.
Sedan gör du bilderna till bakgrund i respektive div och sätter en padding som är lika stor som bildens höjd. Inuti de divvarna ska du inte ha massor med andra divvar. Listor ska vara listor och paragrafer ska vara paragrafer,
... vill du ha enkla radbyten kan du använda
.
Hoppas det klarnar.
|