To generalnie do Adama, ale piszę tu dla wszystkich potencjalnych developerów Coyote.
W kodzie jest wiele wystąpień metody live
. Powinniśmy tego unikać i zamiast tego stosować delegate
, gdyż live
leci po wszystkich elementach aż od document
, zamiast od rodzica interesujących nas elementów. Live było pierwsze, delegate niweluje jego ułomności i generalnie live powinno być już nieużywane.
Dodatkowo ponieważ live()
bazuje na bąbelkowaniu w górę, nie ma możliwości zatrzymania zdarzenia poprzez e.stopPropagation()
module/forum/template/js/posting.js
dla przykładu zawiera aż 16 live
, a to tylko jeden plik. W wolnym czasie można by to przepisać.
PS. I druga wtopa, którą zauważyłem. Adamie, dodawaj var
przed zmiennymi lokalnymi. Widziałem już wiele problemów przez używanie globali...