Recensioner för den här sidan
AL
Publicerad Friday 12th February 2010
- 0 tummen upp.
- 0 tummen ner.
Designen
Jag tycker designen ser bra ut. Clean och funktionell med en del sköna detaljer. Inget att klaga på men det känns ändå som att något saknas. Något som får den att sticka ut mer... jag vet inte riktigt. Det blåa i headern - har du provat med vitt istället? Tycker det skär sig lite med bilden som det är nu...
Koden
Koden ser också fin ut. Generellt semantiska element, IDn och klasser. Här är några saker jag hade gjort annorlunda dock:
- Lägg allt ditt Javascript i botten av sidan, precis innan </body>. Då nästan alla script körs "body.onload" ändå så finns ingen anledning att inkludera dem innan body har loadat.
- title-attribut med samma värde som själva länken är lite onödigt. En skärmläsare läser upp "Prisinformation Prisinformation".
- Att hoppa från h1 till h3 är ogiltigt. Jag vet att validatorn inte gnäller men jag är rätt säker på att det står i specen.
- Jag hade hellre använt en DL för öppettiderna, men om du ska köra tabell så använd även th, och i ditt fall måste du speca scope-attributet till "row" också. Då förstår en skärmläsare att "Vardagar" är kopplat till "06:00-00:00".
- Det är hyffsat enkelt att "self-clear:a" floatade element med ren CSS. Oftast funkar overflow: hidden; på det omslutande elementet, annars finns andra lösningar om du googlar "clearfix" eller nåt. Du kan mao skippa alla div.clear-divar.
- I footer använder du left och right-namn. Jag hade personligen kört :first-child och :last-child istället men om IE-stöd är ett krav så hade jag gett dem first/last-child klass-namn istället. Det är bättre då det inte är designrelaterat som left och right. I det här fallet hade jag nog iofs bytt ut den första listan (som egentligen inte är en lista - då den bara innehåller en punkt) till en paragraf.
Overall bra jobbat tycker jag!
Kommentera den här recensionen
Kommentarer på den här recensionen
7
4
2
8
2
a
T
2
9