Proszę o ocenę kodu:

http://studiologika.pl/stalowemiasto/Pocieta/

1. Ikonki w nawigacji - bardzo fajnie zrobione, jako jeden sprite, za to +
2. Stanowczo za dużo divów - połowa z nich powinna być paragrafami W całym kodzie nie ma ani jednego paragrafu, szok!
3. Zaokrąglenia: mogłeś to rozwiązać za pomocą border-radius + jakiś skrypt pod IE, np. dd_roundies, Twój sposób z 3 divami + span w środku jest mega nieoptymalny, jeżeli chciałeś to zrobić za pomocą grafiki to wystarczył by div w divie
4. Układ - od strony SEO sidebar powinien być w kodzie za contentem, także to popraw
5. za dużo klas, za mało id - takie rzeczy jak header, sidebar, content, nav, footer - to powinny być id.
6. W menu z pogodą - w odsyłaczach jest span, który nic nie robi - po co?
7. Dużo obrazków nie ma ustalonych wymiarów
8. Katalog firm - li.break zbędne
9. Stopka: | po prawej stronie powinno być jako tło

Ogólnie: strona całkiem przyzwoicie zakodowana, jeżeli nie liczyć OGROMNEGO potknięcia z divami - radzę poczytać o divitis

tutaj akurat się nie zgodzę - sam wyznaje zasadę: klasy dla css, id dla js

a tak to zmienić kolor tekstu na żółtym tle (biały to trochę zły wybór ) i strona jak najbardziej ok - bez oczopląsy oraz łatwa w nawigacji

A wiesz że w html5 zmniejszyłbyś kod o ok 50% ? komentarz warunkowy dla IE6 z alertem? Zamień to na div z informacją u góry. Resztę dobrze podsumował Pawel_W.

1. Wg mnie strasznie dużo kodu
2. Strona się wczytuje 1-3sek przy prędkości Internetu 40mb/s
3. Do czego mają służyć te kropki w menu po lewej? Myślałem że można sortować a tu nic

<script type="text/javascript" src="js/jquery/jquery.1.6.4.min.js"></script>

"If you’re doing this on a public facing website, you are doing it wrong."

http://encosia.com/3-reasons-why-you-shoul...jquery-for-you/