jQuery plugin

[ryuran]

La version violente en mode plugin jQuery et programmation objet. Il y a encore des amélioration à faire principalement pour les ID

• [ ] edit readme.md about config and markup

[nico3333fr]

Hello,

merci encore pour ce refacto violent mais salutaire :)

Je suis en train de tester tout cela à vue de nez. Voilà ce que je note :

• tu as changé la structure volontairement ? Avant c'était :

<div class="js-accordion">
 <h2 class="js-accordion__header">blablabla</h2>
 <div class="js-accordion__panel">contenu</div>
 etc.
</div>

Maintenant, c'est :

<div class="js-accordion">
 <div class="js-accordion__panel"><h2 class="js-accordion__header">blablabla</h2>
  Contenu
 </div>
 etc.
</div>

Vois-tu une raison bloquante pour garder l'ancienne structure ? (c'est juste que si on peut éviter de trop péter l'existant, ce serait mieux :) )

• il manque une option importante (pour le côté facile à déployer) : les classes générées pour le stylage. En fait, l'attribut data-accordion-prefix-classes="minimalist-accordion" permet de générer un namespacing et une batterie de classes genre minimalist-accordion__header etc. qui permettent d'avoir plusieurs accordéons stylés différemment en utilisant le même module (cf https://a11y.nicolas-hoffmann.net/accordion/ (option, "wanna see it animated"). Est-ce qu'on y met dans la config de base ?

(je vais faire d'autres essais)

Au plaisir, Nico

[ryuran]

Oui, j'ai changé volontairement la structure.

• Car le js déplaçait le titre, modifier l'arbre DOM a un coût en perf.
• Pas de raison particulière pour qu'il ne soit pas dés le départ dans le <div>.

Pour les classes et le data-accordion-prefix-classes="minimalist-accordion", le plugin ne devrait pas imposer le namespacing des classes aux développeur. Les sélecteurs étant configurables il utilise les classes qu'il veut.

Pour ce qui est de la css d'animation, il suffis d'adapter les selecteurs css.

[nico3333fr]

Yop,

juste un message pour te dire que je n'ai pas abandonné : je progresse lentement (c'est mon côté suisse), mais ça commence à venir. J'ai corrigé un petit bug et je vois pour le reconnecter avec mon existant.

Je te tiens au jus :) Nico

P.S : par contre, j'insiste pour que tu ne recodes pas le simple expand, ça sera mon exercice pour vérifier que j'ai compris ce que tu as fait !

[ryuran]

J'ai un doute dans mon code sur l'usage de e.target vs e.currentTarget il est possible que je me soit trompé là dessus.