[JQuery] (load clik toggle) Czy da się to uprościć? Opcje

[KotWButach]

Witam
Mam kod i zastanawiam się czy i jak można by go skrócić czy możecie pomóc?

[PHP] pobierz, plaintext 
    $('.table-component').on('click', '.filters', function() {
        $('.menu-filters').load('load/test.html'); // menu
    });
 
    // ukrycie
    $('.menu-filters').hide();
 
    // Obsłużenie kliknięcia
    $('.table-component').on('click', '.filters', function(event) {
        event.stopPropagation();
        $('.menu-filters').toggle();
    });
 
    // schowanie
    $(document).click(function() {
        $('.menu-filters').hide();
    });
[PHP] pobierz, plaintext 

Ten post edytował KotWButach 3.07.2014, 14:51:58

[proszek]

Wszystko co da się zrobić za pomocą CSSa powinno się nim robić - to tak na przyszłość. Optymalnie w JS dodajesz tylko albo odejmujesz odpowiednie klasy.

[SmokAnalog]

Wszystko co da się zrobić za pomocą CSSa powinno się nim robić - to tak na przyszłość. Optymalnie w JS dodajesz tylko albo odejmujesz odpowiednie klasy.

W tym przypadku mam inne zdanie. Po co bałaganić CSS kolejnymi klasami, kiedy jedyną różnicą jest widoczność elementu? To jest jedna (a może jedyna?) z tych właściwości, które lepiej zdają egzamin jako inline style.

Ten post edytował SmokAnalog 4.07.2014, 23:23:14

[pedro84]

W tym przypadku mam inne zdanie. Po co bałaganić CSS kolejnymi klasami, kiedy jedyną różnicą jest widoczność elementu? To jest jedna (a może jedyna?) z tych właściwości, które lepiej zdają egzamin jako inline style.

I wtedy zaśmiecisz HTML. Struktura dokumentu CSS naprawdę nie ucierpi jeśli dodasz do niej dwie klasy: .hidden, .visible. A style inline są zawsze "be".

[SmokAnalog]

A style inline są zawsze "be".

To twoje zdanie (IMG:style_emoticons/default/wink.gif) Ja trzymam się swojego zdania na temat display: none. Nie podobają mi się klasy typu "hidden", bo to tylko prosty tymczasowy stan, a nie jakaś rodzina właściwości. Ale to już dyskusja filozoficzna, każdy ma swoje zdanie.

[pedro84]

To twoje zdanie (IMG:style_emoticons/default/wink.gif) Ja trzymam się swojego zdania na temat display: none. Nie podobają mi się klasy typu "hidden", bo to tylko prosty tymczasowy stan, a nie jakaś rodzina właściwości. Ale to już dyskusja filozoficzna, każdy ma swoje zdanie.

Klasa ma zasadniczy jeden argument "za". Co w przypadku n elementów? Wstawiasz wszędzie style inline?

Poza tym, to nie odnosi się tylko do tymczasowego stanu - to zwykła klasa helpera i tyle.

[SmokAnalog]

Klasa ma zasadniczy jeden argument "za". Co w przypadku n elementów? Wstawiasz wszędzie style inline?

Tak, dokładnie to. Nie widzę problemu.