Attention deux bugs (bénins) ici
[http://zone.spip.org/trac/spip-zone/changeset/21883] :
1) tu as supprimé crayons.js, puis tu l'as recréé : du coup on perdait
l'historique... on ne sait même pas si la version que tu as recréée
est identique ou pas.
2) il ne faut pas mettre d'accents dans le code
je viens de corriger ces deux défauts, le 1) en recopiant crayons.js
depuis une version antérieure, le 2) en modifiant le fichier.
J'ai aussi une critique du même ordre sur
[http://zone.spip.org/trac/spip-zone/changeset/21861] : en remplaçant
tous les espaces par des tabs, tu as masqué une partie de
l'historique, et on ne voit pas le "diff" de ce que tu as introduit.
* masquage : si on cherche quel commit a introduit telle ligne, il
faudra pas mal bidouiller pour remonter, car "svn blame" trouvera
toujours 21861.
* confusion : si on veut relire ton commit, il faut relire 400 lignes,
alors qu'il en fait peut-être 3. (d'ailleurs il en fait effectivement
trois, cf. [http://zone.spip.org/trac/spip-zone/changeset/21907] où
j'ai remis la toute dernière version de crayons.js comme modif de la
version historicisée).
Dernier point, tant qu'on est aux espaces et virgules : en modifiant
le prototype de la fonction $.prototype.cfgCrayons.prototype.iconclick
= function(c) => en function(type, c) , tu as cassé un éventuel appel
externe à cette fonction ; il vaudrait mieux écrire function(c, type)
-- Fil