Po pierwsze dziura XSS -
http://dajbuziaka.cba.pl/index.php?id=1&am...4)%3C/script%3EPo drugie używaj this w jQuery - działa to dużo szybciej
Przykład:
Kod
$('#cal').mousemove(function(e){
$(this).css({'background':'url(img/4.png)'});
});
Po trzecie grupuj css
Kod
a:hover, #ka:hover, #ka2:hover
{
text-decoration:underline;
}
4. Pusty znacznik script na dole strony. Po co?
5. Znacznik center
6. Unikaj parametru style. Przerzuć kod css do pliku szablonów
7. Nie wolno używać kilka <br/> obok siebie. To jest znacznik do łamania lini nie do tworzenia marginesów. Użyj margin
8. Kod js wrzuć do pliku js
9. Nie ma semantyki tj. nagłówków, akapitów. Wszystko oznaczasz jako div - tak się nie robi
10. Zerowa typografia.
@Kildyt: po to aby twórca się czegoś nowego nauczył? Nie chodzi o to aby oceniać prace super wymiataczy bo oni naszej opinii i tak w gruncie rzeczy nie potrzebują i dodają prace tylko dla lansu =).