Discussion :

[Tsukaniki]

Alors, je vous expose mon problème...
voici le code html sur lequel est ciblé mon script :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
<div class="news"><h3><big>GRANDE NOUVELLE !</big></h3><p class="corps">bla bla bla <span class="spoil" onClick="gere_spoil(this);">Spoil</span><div class="spoil"> rebla bla bla bla <small>bla bla toujours</small></div><br />
<br />
fin du bla bla</p><span class="auteur">Tsukaniki </span><span class="date"> le 18-06-2007 à 14:32:16</span></div>

Et voici donc le code javascript de la fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
var spoilUncovered = false ;
function gere_spoil(elem) {
         div_parent = elem.parentNode ;
         i_max = div_parent.childNodes.length ;
         for (i=0 ; i < i_max-1 ; i++) {
             if (div_parent.childNodes[i] == elem) {
                var colP = (spoilUncovered)?('#FFFFFF'):('#000000') ;
                spoilUncovered = !spoilUncovered ;
                div_parent.childNodes[i+1].style.color = colP ;
                for (j=0,j_max=div_parent.childNodes[i+1].childNodes.length ; j<j_max ; j++) {
                   if (div_parent.childNodes[i+1].childNodes[j].tagName != undefined) {
                      div_parent.childNodes[i+1].childNodes[j].style.color = colP ;
                   }
                }
                break ;
             }
         }
}

Le problème est simple : cette fonction ne marche pas... Or sur un autre de mes site, elle fonctionne parfaitement, et la seule différence entre les deux sites sont les balises.
Dans ma recherche de la cause du problème, j'ai découvert que div_parent.childNodes.length vaut 2 . . . ce qui ne devrait pas être le cas au vu du code. Quelqu'un sait pourquoi ?

[SpaceFrog]

une petite recherche avec nodeCleaner

[Tsukaniki]

Hmm je suis peut-être stupide mais un nodeCleaner est sensé supprimer des nodes vides ("rajoutés" par FF), or le problème que j'ai est que seul 2 nodes fils sont trouvés au lieux de 7, donc en supprimer n'aiderai en rien... Sauf si un noeud de valeur null marque une fin de tableau ?
De plus, j'ai testé en ajoutant un nodeCleaner justement, et le résultat est le même...
Je vous remet le code avec le nodeCleaner

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
var notWhitespace = /\S/;
 
var TEXT_NODE;
try {
   TEXT_NODE = Node.TEXT_NODE;
} catch(e) {
   TEXT_NODE = 3;
}
 
var ELEMENT_NODE;
try {
  ELEMENT_NODE = Node.ELEMENT_NODE;
} catch(e) {
   ELEMENT_NODE = 1;
}
 
function cleanWhitespace(node) {
  for (var x = 0; x < node.childNodes.length; x++) {
    var childNode = node.childNodes[x];
    if ((childNode.nodeType == TEXT_NODE)&&(!notWhitespace.test(childNode.nodeValue))) {
      node.removeChild(node.childNodes[x]);
      x--;
    }
    if (childNode.nodeType == ELEMENT_NODE) {
      cleanWhitespace(childNode);
    }
  }
}
var spoilUncovered = false ;
function gere_spoil(elem) {
         div_parent = elem.parentNode ;
         cleanWhitespace(div_parent) ; // Si j'ai bien tout compris, c'est bien ici qu'il faut le mettre non ?
         i_max = div_parent.childNodes.length ;
         window.alert('i max vaut : '+i_max) ;
         for (i=0 ; i < i_max-1 ; i++) {
             if (div_parent.childNodes[i] == elem) {
                var colP = (spoilUncovered)?('#FFFFFF'):('#000000') ;
                spoilUncovered = !spoilUncovered ;
                div_parent.childNodes[i+1].style.color = colP ;
                for (j=0,j_max=div_parent.childNodes[i+1].childNodes.length ; j<j_max ; j++) {
                   if (div_parent.childNodes[i+1].childNodes[j].tagName != undefined) {
                      div_parent.childNodes[i+1].childNodes[j].style.color = colP ;
                   }
                }
                break ;
             }
         }
}

[SpaceFrog]

le decompte des childNodes prend en compte également les petits enfants ...

remplace ta balise p par un div et tu auras le nombre total de childNodes ...
y compris les textNodes...

il semblerait que la balise p ne tienne pas compte des textNodes...
étant plutot une balise de mise en forme de texte plutot qu'une balise sonteneur d'objet cela ne parait pas idiot ...

[Tsukaniki]

Oui, je me suis rendu compte que le problème venait surement de la balise p, mais je souhaitais contourner le problème pour éviter de refaire la mise en forme de mon site.
Je me suis rendu compte au fil de mes tests que la nodelist des noeuds fils d'un p s'arrête au dernier noeud inline...
Donc apparemment pas de solutions, faut juste éviter ce cas de figure :/

Merci pour les réponses en tout cas