Ja, nu har jeg ikke læst det hele grundigt igennem, men gennemgået hovedtrækkene..

Jeg synes generelt opbygningen virker lidt rodet..

Afsnit 1) - Udmærket

Afsnit 2) - Også fin

Afsnit 3) - her synes jeg så vogenen kører af sporet, synes det er vigtigere at du gennemgår hvordan man opretter sin egen css-fil og hvordan man får den implementeret på sin side, og evt. laver nogen eksempler man selv kan teste.. Det du har i afsnit 3) er for mig irrelevant viden hvis jeg lige er startet på css..

Afsnit 4) - Så jeg gerne sat som afsnit 3.. Desuden synes jeg det er fint du gennemgår de forskellige måder at lave css på på internt og eksternt, synes dog personligt at den interne måde er lidt spildt, da den "rigtige" måde at lave det på er eksternt.. (Du har dog stadig ikke specificeret hvordan man får filen implementeret (kommer i afsnit 5, ved jeg men synes de hører sammen)

Afsnit 5) - Bør slås sammen med 4) imo

Afsnit 6) - Inline CSS bør du ikke gennemgå, da det er "forkert", det ville være dumt at benytte sig af denne metode set i forhold til at du allerede har gennemgået hvordan man laver i en ekstern fil..

Afsnit 7) - du viser hvordan man opretter en class på "p" attributten, synes næsten det er vigtigere du lærer folk at ændre generelt på p, h1, h2, h3 m.v. da classes ikke er noget man benytter sig af særlig ofte, imo .. Du bør desuden gennemgå <span> teknikken til at implementere classes på..

Lige en hurtig gennemgang .. Håber du kan bruge det til noget, alt er ment som konstruktiv kritik, og forsøger på ingen måde at nedgøre dit arbejde, da jeg ved der ligger hårdt arbejde bag den!

tjaeh... har et par kommentare til din CSS:
1) det er 'good practice' kun at bruge lowercase i CSS, dine eksempler svinger lidt mellem store og små bogstaver, og mens det er fuldt lovligt, er det ikke kønt
2) du lader ikke til at forklare hvorfor det ikke er smart at bruge inline css - hvilket er en ret vigtig pointe
3) du opkalder dine classes efter deres udseende, hvilket er ret usmart - hvis du senere ændrer på din CSS (hvilket jo er et af hoved-argumenterne for at bruge CSS i det hele taget), så kan du ende med en side hvor der står <p class="hvid">foobar</p> men hvor teksten er noget helt andet end hvid... - navngiv altid udfra indhold eller funktion - ikke udseende
4) din HTML er invalid - måske ikke enormt smart i en guide som denne (du har f.eks. ikke nogen doctype, du har tekst uden en container (f.eks. <p>) og så bruger du <br> inkorrekt)


har kun skimmet den, så det kan godt være jeg egentlig har flere kommentare, men er på arbejde, så de må i så fald komme senere...

#1 Mange tak, og tager alt til efterfølge :o)

#2 Okaay, men mht. design og sådan det er stillet op, så bliver dette ændret så der er valid... :o)

Men tak til begge :D

#3 når jeg siger "din HTML er invalid", mener jeg den der vises i guiden, ikke den du selv bruger på din side (den har jeg ikke tjekket)

og med hensyn til #1, så vil jeg derimod mene at classes er en MEGET vigtig del af CSS, men du bør nok skrive om det i sammenhæng med ID's og måske en smugle om specificity - selvom dette ikke er en nemt emne (man kunne evt. bare nævne at det fandtes, og komme ind på det senere)

#4 - Nej selvfølgelig er classes vigtige, heller ikke ment på den måde. Dog benytter jeg mig mere af bare at ændre på p, h1, h2, h3 m.v. end at benytte mig at classes, disse bruger jeg dog til forskellige ting også, hvis der skal fremhæves ord, tekst-stykker, understregning af links (hvor understregningen skal have en anderledes farve) m.v.

#0
OT: Hvordan laver du den der skiftende baggrundsfarve på din hjemmeside, når man scroller op og ned?