30-01-2011, 01:42
1) Pierwsze co się rzuca w oczy to min-width; które w twoim przypadku jest bez sensowne. Skoro cała strona oparta jest na procentach, to czemu min-width na px? Stąd rozwala ci się layout.
2) Osobiście też zrezygnował bym z nazewnictwa kolorów na rzecz ich wartości hex lub rgb.
3) Dawno nie widziałem strony w której wykorzystywane jest h5 i h6. Baaardzo dawno. Ogólnie kończy się na h3, a reszta to strong lub podobne - zależnie od sytuacji. U ciebie np <h6 id="newsautor"> jest błędem, bo autor nie jest "tytułem".
4) Ah. I najgorsza z możliwych rzeczy
Używasz ID zamiast Class. Jakby to było dobrze użyte - nie czepiał bym się, ale jest błędnie. Dane ID może być tylko 1 raz na stronie!
Reszty grzechów nie zauważyłem, ale nie patrzyłem zbyt dokładnie.
2) Osobiście też zrezygnował bym z nazewnictwa kolorów na rzecz ich wartości hex lub rgb.
3) Dawno nie widziałem strony w której wykorzystywane jest h5 i h6. Baaardzo dawno. Ogólnie kończy się na h3, a reszta to strong lub podobne - zależnie od sytuacji. U ciebie np <h6 id="newsautor"> jest błędem, bo autor nie jest "tytułem".
4) Ah. I najgorsza z możliwych rzeczy
Używasz ID zamiast Class. Jakby to było dobrze użyte - nie czepiał bym się, ale jest błędnie. Dane ID może być tylko 1 raz na stronie!Reszty grzechów nie zauważyłem, ale nie patrzyłem zbyt dokładnie.
