Zarábaj až 6 000 € mesačne! Akreditované rekvalifikačné kurzy od 0 €. Viac informácií.

Diskusia – Jednoduchá Collapsible v JavaScripte s knižnicou jQuery

Späť

Upozorňujeme, že diskusie pod našimi online kurzami sú nemoderované a primárne slúžia na získavanie spätnej väzby pre budúce vylepšenie kurzov. Pre študentov našich rekvalifikačných kurzov ponúkame možnosť priameho kontaktu s lektormi a študijným referentom pre osobné konzultácie a podporu v rámci ich štúdia. Toto je exkluzívna služba, ktorá zaisťuje kvalitnú a cielenú pomoc v prípade akýchkoľvek otázok alebo projektov.

Komentáre
Avatar
Honza Bittner
Tvůrce
Avatar
Honza Bittner:4.3.2015 13:43

Inu, dalo by se to udělat lépe.

  1. Obecně doporučuji všechno co souvisí s CSS dávat do CSS. V tvém případě to znamená, že nebudeš muset animovat změnu rotace v JS, ale jen změníš třídu a CSS si to vyřeší samo.

Je možné, že ti to kvůli tomu půjde jen v CSS3, ale řekněme si to narovinu - kdo dnes nemá prohlížeč s podporou CSS3 (?) a je opravdu nutné, aby se animace provedla i ve starých prohlížečích?
http://caniuse.com/#…

  1. Když už využíváš jQuery knihovnu, využívej jí na plno. Tedy např. pro tvůj kód níže:
if (collapsible.hasClass('aktivni-collapsible') == true) {
        collapsible.slideUp();
        AnimaceOtoceni(collapsible_fa, 360, 180);
        collapsible.removeClass('aktivni-collapsible');
    } else {
        collapsible.slideDown();
        AnimaceOtoceni(collapsible_fa, 180, 360);
        collapsible.addClass('aktivni-collapsible');
    }

Kód se dá perfektně zminimalizovat na 2 řádky.

collapsible.slideToggle();
collapsible.toggleClass('aktivni-collapsible');
  1. Odeber události z HTML a raději vytvoř jQuery handlery. Kódy tím tak separuješ a vytvoříš lepší strukturu.
jQuery(".trida").on("click", function(){
  ... obsah ...
});
  1. Uvádíš import FA zapsán způsobem níže, ale jsem si jistý, že ti tam chybí http:, nebo ne? :)
<link rel="stylesheet" href="//maxcdn.bootstrapcdn.com/font-awesome/4.3.0/css/font-awesome.min.css">
Editované 4.3.2015 13:45
Odpovedať
4.3.2015 13:43
FIT ČVUT alumnus :-) Sleduj mě na https://twitter.com/tenhobi a ptej se na cokoli na https://github.com/tenhobi/ama.
Avatar
Patrik Smělý
Tvůrce
Avatar
Odpovedá na Honza Bittner
Patrik Smělý:4.3.2015 13:54

Děkuji za tvůj názor

  1. v tom máš pravdu, ale když je to v sekci javascript tak jsem to chtěl udělat v javascriptu.
  2. na to platí 1.
  3. v tom máš pravdu.
  4. ne je to dobře ukázku že je to funkční je zde view-source:http://sogocze.cz/ (11 řádek).

Příště se vše pokusím řešit lépe :).

 
Odpovedať
4.3.2015 13:54
Avatar
Patrik Smělý
Tvůrce
Avatar
Odpovedá na Honza Bittner
Patrik Smělý:4.3.2015 16:30

Ještě oprava, nějak jsem přehlídnul to toggle. No uznávám že toggle by možná bylo lepší řešení.

 
Odpovedať
4.3.2015 16:30
Avatar
Vít Cigánek
Člen
Avatar
Odpovedá na Honza Bittner
Vít Cigánek:3.12.2015 18:43

Mi přijde, že je tam hodně kodu less s scss navíc exportovaný font... Musí tam všechno být, když chci udělat jenom rotaci šipky a rozklik toho článku.

 
Odpovedať
3.12.2015 18:43
Robíme čo je v našich silách, aby bola tunajšia diskusia čo najkvalitnejšia. Preto do nej tiež môžu prispievať len registrovaní členovia. Pre zapojenie sa do diskusie sa zaloguj. Ak ešte nemáš účet, zaregistruj sa, je to zadarmo.

Zatiaľ nikto nevložil komentár - buď prvý!