mała strona w php Opcje

[emanes]

witam nie wiedzialem jak nazwać temat wieć nazwałem go tak. Jest to strona która zrobiłem dzisiaj w jakies pare godzin prosze o ocene. link
Strona

[SHiP]

Po pierwsze dziura XSS - http://dajbuziaka.cba.pl/index.php?id=1&am...4)%3C/script%3E
Po 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 =).

Ten post edytował SHiP 15.06.2010, 12:59:42