QUOTE (Rosco @ mercredi 23 mai 2007 à 21:44:50) :
Je le sais évidemment
, tout sera mis ds un seul JS + tard, c'est ds mon TODO depuis des mois, mais j'ai pas que ça à faire pour l'instant... Ca ne changera qu'une infime fraction du poids de la page qui est gzippée de toute façon, mais ça sera fait quand même car autant alléger.
Le JS est bien, le tien était en + faux y me semble, celui de l'autre fois. On s'en tape de tes événements qui servent à rien à part alourdir
Euh, le js que j'ai posté fonctionnait et fonctionne parfaitement, je te donne juste deux hints :
-Un retour de ligne est un node, donc ta méthode sent des fesses.
-Le <br /> est un node aussi.
Le mieux pour pas se gourrer est de matcher le className via regexp
Par contre passer (this.childNodes[1]) en paramètre y'a rien de plus bourrin, ensuite si tu savais faire du js tu saurais que pour redonner sa valeur d'origine à une propriété (en l'occurence le style.display) on utilise style.display=''.
Et avec les évenements t'évites de foutre un onclick sur chaque spoiler, ça alourdit rien, bien au contraire. Ca te permettrait même de démontrer que t'es pas complètement nul en te documentant sur la propagation et en proposant le premier forum avec des spoilers multiples (Par forcément utile, mais tellement marrant
).
Message édité par Shinuza le mercredi 23 mai 2007 à 22:21:54