Wyłączenie click dla elementów

Question

Cześć, posiadam skrypt, który powinien najpierw wyhaczyć wszystkie divy z klasą .items, gdzie klasa .price jest mniejsza od 1, dodać im parę elementów css, oraz zabraniać używania na nich funkcji click. Chodzi o to, że w dalszej części skryptu po naciśnięci na diva z klasą .items jest on usuwany z miejsca w którym stoi i przenoszony do takiego "koszyka". Niestety, ale funkcja off w ifie nie działała. Macie jakieś pomysły?

Kod:

http://pastebin.com/aJT1UPJg

Answer 1

Na początek polecam normalnie formatować kod, bo połapać się w nim jest niesamowicie trudno.

Co do problemu: a wywołanie $( this ).off( 'click' )?

Comments

Niestety nie działa :/ Kolor i kursor jest zmieniany, ale można diva kliknąć, wykonuje się potem to samo co z resztą divów.

---

Co dokładnie masz na myśli mówiąc formatować?

---

if (value<1) {$(this).css("color", "red").css("cursor", "not-allowed");
           }

vs

if (value<1) {
    $(this).css("color", "red").css("cursor", "not-allowed");
}

Który jest czytelniejszy?

Co do kodu: tak w sumie, to przecież nie ma żadnego $.fn.off w if-ie.

---

Trochę go uporządkowałem, a co do tego co napisałeś, to nie wiem czy o to ci dokładnie chodziło. http://pastebin.com/7ALQccfi Teraz nie można kliknąć niczego.

---

Chodziło mi o:

if (value<1) {
            $(this).css("color", "red").css("cursor", "not-allowed");
            $( this ).off( 'click' );
 }

 

---

Też próbowałem i niestety nie.  http://pastebin.com/RRgv68XC

---

Bo za tym ifem podpinasz klik. Klik powinien być podpięty tylko wówczas, gdy value >= 1 (wówczas można się w sumie pozbyć off).

Answer 2

Dobra, udało mi się to rozwiązać, dodałęm return 0; zamiast off i działa. Napiszcie, czy na pewno jest w porządku i czy takie rozwiązanie jest dobre :D http://pastebin.com/Cyz4xTYq