Erreur sur la ligne de la boucle

Version imprimable

15/08/2016, 15h57
patricktoulon

Erreur sur la ligne de la boucle
bonjours a tous
j'ai une erreur sur la boucle "for"

ne riez pas ca fait plus de 3 heures que je cherche l'erreur
le message me donne bien le length donc la collection des "shado" est bien pris en compte
par contre quand je boucle dessus rien a faire ca plante
Code:

1 2 3 4 5 var elements =document.getElementsByClassName("shado"); alert(elements.length); for (var i =0; i < elements.length; i++) { alert( elements[i].innerHTML); }
pitié je vais faire une rupture la !!!!
15/08/2016, 16h09
patricktoulon

re

ne prenez pas en compte la suite"blabla 8203..." il se met tout seul dans la balise code a la fin
encore un petit bug dan le wysiwyg dvp
16/08/2016, 01h51
Invité

Bonjour,

Non ce n'est pad un bug DVP : tu as certainement un caractère invisible apres }
16/08/2016, 08h28
patricktoulon

Re

Re
Bonjour
Si si c est bien une erreur d interpretation
Essaie de taper. Cette expression. Entre guillemets dans ta reponse
Et d envoyer ta reponse ce qu il y a entre guillemet aura disparu
C est donc un bug. Ou tout du moins expression interdite
Bon c est pas trop grave j aurais préféré une reponse a. Mon soucis que que n ai toujours Pas pigé
Merci pour le. Retour
16/08/2016, 08h57
SpaceFrog

Le souci provient d'une partie que tu ne nous montre pas ...

La boucle fonctionne parfaitement
https://jsfiddle.net/kwfq714a/
16/08/2016, 09h12
Invité
Citation:

Envoyé par patricktoulon

ne prenez pas en compte la suite"blabla 8203..."

Justement SI.
C'est là le problème.

Si tu regardes ton 1er message, tu verras que j'ai corrigé, et réussi à SUPPRIMER l'affichage du caractère "#8203" !
=> en supprimant un caractère invisible :
- "#8203" est un caractère unicode, sans largeur d'espace (on ne le voit pas !)
Donc :
- soit tu copies-colles le script de ton 1er message (puisque je l'ai corrigé ;))
- soit tu effaces tout le bout de code dans ton script, et tu le ré-écris A LA MAIN (PAS par copier-coller)
N.B. Et j'ai bien répondu à la question...

Citation:

Envoyé par patricktoulon

encore un petit bug dans le wysiwyg dvp

As-tu une idée du nombre de messages et codes rédigés via ce WYSIWYG ?
Les "bugs" passent rarement inaperçu, et n'attendent jamais longtemps avant d'être corrigés... (quand ce sont vraiment des bugs...) :aie:

Attention : "La Vérité est ailleurs" ne veut pas dire "c'est forcément la faute de l'Autre"...
16/08/2016, 11h56
SpaceFrog

Citation:

La Vérité est Tailleur

C'est pas une citation de Coco Chanel ?

re
rebonjour spacefrog

ceci etant réglé aurais tu une solutions
j'ai beau chechercher je ne trouve rien j'ai fait plein d'essais et toujours rien

voila la fonction complete

Code:

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
 
/////////////////////////////////////////////////////////////////////////////////
function selectionne_text()
   {
 
return window.getSelection().getRangeAt(0);
   }   
 
////////////////////////////////////////////////////////////////////////////////
////////////////////////////////////////////////////////////////////////////////////
function shadowcolor(coul){
if (selectionne_text()){   
                               var selection = selectionne_text();
   var selectedText = selection.extractContents();
   var balisefont = document.createElement("FONT");
   balisefont .className="shado";
                                             balisefont .appendChild(selectedText);
 
                                             balisefont .style.textShadow = "0px 0px 10px"+coul; 
    selection.insertNode(balisefont);
alert(balisefont.parentElement.innerHTML);
//var elements =document.getElementsByClassName("shado");
//alert(elements.length);
//for (var i =0; i < elements.length ; i++) { alert( elements[i].innerHTML);}
}
}
///////////////////////////////////////////////////////////////

en fait par la suite je veut boucler sur tout les shado et tester si le parent élément est un shadoalors j'applique le même shado et remplace la balisefont par son innertext dans le parent avec un string replace

a moins que tu connaisse le moyen de connaitre le fontfamily,size,etc de la selection

16/08/2016, 14h55
SpaceFrog

Code:

balisefont .className="shado";

ils y sont les espaces avant le point dans ton code ?
16/08/2016, 15h12
patricktoulon

re

re
oui je peut modifier cela si tu veux bien que la le msgbox(alert en javascript) après me donne bien le nombre d'élément donc ca fonctionne

je reviens se soir je repart au boulot
16/08/2016, 15h14
Invité

Surtout ne dis pas "merci"... :roll:
16/08/2016, 19h50
patricktoulon

Re

Bonsoir heu.... Oui si tu veux mais. Merci pourquoi
Mon soucis n est toujour pas résolu,

L erreur sur le wiziwig dvp n etait pas le sujet du post

Alors quand. Meme merci;)
16/08/2016, 23h40
patricktoulon

re

re
a bon !! ???????

merci a tous ce qui m'ont répondus même si le sujet n'est pas résolu

surtout spacefrog

Ps spacefrog le code html est dans un textbox multi ligne dans un userform et les espaces n'ont aucunes incidence c'est comme si j'avais écris le code avec Notepad

je met ce post en résolu quand même je trouverais bien un jour
17/08/2016, 12h25
melka one

jour

il manque un espace et aussi px a la fin de cette ligne

Code:

balisefont.style.textShadow = "0px 0px 10px "+coul+"px";
17/08/2016, 12h45
patricktoulon

re

bonjour Melka one
ok pour l'espace mais le "px" il y a méprise coul est une couleur

et l'erreur était dans la ligne "for" a la base

merci pour ton intervention;)
17/08/2016, 15h02
melka one

autant pour moi

mais que retourne la boucle pour chaque itération
17/08/2016, 20h04
patricktoulon

re

re
ben justement c'est elle qui déclenche l'erreur et impossible de comprendre pourquoi

mon but étant après application de textshadow sur la balise font ajoutée dynamiquement
controler si le parent de cette balise a le même intertexte ce qui veut dire que je peut appliquer le textshadow au parent et ne garder que le innertext de la balise ajoutée

il y a certainement mieux en terme de méthode mais j'ai essayé plein de chose qui ne même a rien

merci pour les retours
17/08/2016, 20h44
Invité

Citation:

Envoyé par patricktoulon

ben justement c'est elle qui déclenche l'erreur et impossible de comprendre pourquoi

Je t'ai donné une explication, mais tu ne sembles pas l'avoir comprise...

re-montre le code, pour voir ?

N.B. et comme je suis sympa (...), je t'explique pourquoi j'ai mis (puis RETIRE) les :moinsser: :
La règle élémentaire de politesse est de dire "Merci" à TOUS ceux qui ont pris le temps de se pencher sur TON problème, et de te répondre *.
Pas uniquement à ceux qui t'ont trouvé LA solution...

(après plus de 7000 messages sur DVP, je pensais que tu l'avais compris...)
A bon entendeur...

* C'est ce qui motive à (continuer à ) répondre... et à créer une bonne dynamique dans la communauté DVP

2 pièce(s) jointe(s)

re
bonjour jreaux62
oui c'est sans doute un truc bête

en fait pour reprendre la question avec plus de précision j'ai un bouton qui appelle la fonction shadowcolor
cette fonction ajoute le textshadow

ca c'est OK ca fonctionne
mais comme faire un wysiwyg sans query n'est pas une mince affaire a moins d utiliser des regex a tour de bras je suis obligé de bidouiller

je précise que je ne peut pas utiliser query parce que mon document (wysiwyg est dans un controls ocx(webbrowser) qui est largement élagué en terme de possibilité par rapport a un explorer quelconque et donc ne supporte pas le query

ceci etant dit
mon idée était de boucler sur tout les elements et tester si le parent ou l'enfant(0)et unique de l'item dans la boucle avait pour innertext la meme chose
si c'est le cas regrouper les deux dans une meme balise("font")

voila mon projet (tout du moins une partie )

voila le code de la boucle
Code:

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 ///////////////////////////////////////////////////////////////////////////////// function selectionne_text() { return window.getSelection().getRangeAt(0); } //////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////// function shadowcolor(coul){ if (selectionne_text()){ var selection = selectionne_text(); var selectedText = selection.extractContents(); var balisefont = document.createElement("font"); balisefont.className="shado"; balisefont.appendChild(selectedText); balisefont .style.textShadow = "0px 0px 10px"+coul; selection.insertNode(balisefont); alert(balisefont.parentElement.innerHTML); var elements =document.getElementsByClassName("shado"); alert(elements.length); for (var i =0; i < elements.length ; i++) { alert( elements[i].innerHTML);}// beepbeep erreur !!!!!!!!!!si je l'enleve l'ombre ce fait bien dans le texte } } ///////////////////////////////////////////////////////////////
je dirais même que cette boucle devrait être effectuer avant la création de la balise afin d'appliquer directement a l'item et sauter la creation

après la création je pense avoir trouver un moyen de tester
car il y a un phénomène intéressant qui se produit

dans mon wisiwyg si je sélectionne du texte et change la police ,change la taille et ajoute le texshadow en une fois, la balise ajouté est l'enfant de la balise initial

par contre si je sélectionne du texte change la taille et police puis désélectionne et re sélectionne la balise ajoutée devient le parent de la balise initiale

j'ai donc fait ceci pour tester
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 //////////////////////////////////////////////////////////////////////////////////// function shadowcolor(coul){ if (selectionne_text()){ var selection = selectionne_text(); var selectedText = selection.extractContents(); var balisefont = document.createElement("FONT"); balisefont.className="shado"; balisefont.appendChild(selectedText); balisefont.style.textShadow = "0px 0px 10px "+coul; selection.insertNode(balisefont); if (balisefont.children.length==1){ if(balisefont.children(0).innerText==balisefont.innerText){ alert("enfant: "+balisefont .outerHTML);} } if (balisefont.parentElement.innerHTML==balisefont.outerHTML){ alert("parent: "+balisefont.parentElement.outerHTML); } } } ///////////////////////////////////////////////////////////////
si il le faut je vous fait une capture annimée pour etre plus clair avec appercu du code html obtenu

voila je sais pas si j'ai été clair sur ce coup la :oops:

merci pour les retours

capture je sélectionne et change 3 propriétés d'un coup
Pièce jointe 218125

maintenant je sélectionne du texte et change d'abords la police e la taille puis dé sectionne et re sélectionne puis met le shadow
Pièce jointe 218126

voila j'espère que les msgbox en diront plus long sur mes intentions

18/08/2016, 12h46
melka one

ce serais pas mieux de testé la présence d'une balise font et si present lui assigné le textShadow
18/08/2016, 19h23
patricktoulon

re

Bonsoir Melka one

ben oui c'est pour ca la boucle

si dans la boucle l'element[i].innertext =selection alors mettre le textshadow a l'élément et sauter la création de la balise font c'est ce que j'ai dis plus haut

en fait il faudrait même détecter le "" parent de la sélection a fin de ne pas mettre le shado sur une autre ligne qui aurait la même chaine de caractères sélectionnée

re
bon après une bonne journée de boulot je cogite mieux va savoir toi

voila j'ai presque obtenu ce que je souhaite sauf que je boucle sur document .all si il y a plusieur chaines identique dans le texte ca risque de mettre le shado sur le mauvais
mais déjà ma boucle fonctionne

voila ou j'en suis

Code:

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
////////////////////////////////////////////////////////////////////////////////////
function shadowcolor(coul){
if (selectionne_text()){   
var selection = selectionne_text();
 
var elements=document.all;
for (var i=0;i<elements.length;i++){  if (elements[i].innerText==selection){var elem=elements[i];var ok="ok"}}//on détermine l'élément et la variable ok si c'est bon 
 
if(ok=="ok"){elem.style.textShadow = "0px 0px 10px "+coul; }//si c'est bon l'élément prends le shado 
 
if(ok !="ok"){ //si c'est pas bon !!!
 var selectedText = selection.extractContents();
   var balisefont = document.createElement("FONT");
   balisefont.className="shado";
                                             balisefont.appendChild(selectedText);
                                             balisefont.style.textShadow = "0px 0px 10px "+coul; 
   selection.insertNode(balisefont);
if (balisefont.children.length==1){
                                                                  if(balisefont.children(0).innerText==balisefont.innerText){
                                                                                                        alert("enfant:  "+balisefont .outerHTML);}
                                                                 }
 
if (balisefont.parentElement.innerHTML==balisefont.outerHTML){
                                                                                                                    alert("parent:  "+balisefont.parentElement.outerHTML);
                                                                                                                    }     
 
}
}
}
///////////////////////////////////////////////////////////////

y aurais il un moyen rapide (expression pour déterminer le parent("") sur la selection ?????? ca serait bien plus pratique

18/08/2016, 20h04
patricktoulon

re

re
autant pour moi c'est bon !!!!!!! j'ai fait le test avec 3 chaines de caractères identiques dans mon texte et le fait de tester selection fait que le shado est bien appliqué sur la chaine que j'ai sélectionné même si c'est pas le premier

youpi tralala la ca marche

il me reste plus qu'a trouver le moyen de supprimer un éventuel texshadow qui serait parmi la sélection

en effet si j'ai monde avec "on" en texte shado vert par exemple et que je sélectionne "monde" ca me met tout ce qui n'est pas en vert avec le shado mais laisse "on" en vert il faut donc que je supprime la parti en vert

en tout cas je pige vite ce soir

bon après il faudra certainement mettre ca au propre voir faire des if else plutôt que des if a la suite

super trop content je vais pouvoir avancer

merci merci !!!
18/08/2016, 20h28
Invité

Bon.... :whistle:

Je ne voudrais pas passer pour le rabat-joie de service, mais la balise est obsolète...

Citation:

Cette fonctionnalité est obsolète.
Bien qu'encore supportée par des navigateurs, son utilisation est découragée pour tout nouveau projet.
Évitez de l'utiliser.

Pour ton WYSIWYG, tu utilises notamment .....

Si je me réfère à CKeditor (que j'utilise), on aurait plutôt 3 balises :
....
18/08/2016, 20h41
patricktoulon

re

Bonsoir Jreaux62

non c'est pas rabajoi
je connais ckeditor il y a aussi la dernière évolution du TINI editor qui est pas mal aussi

seul problème il utilise "query" et malheureusement moi je ne peut pas dans mon userform excel

après des spans pour chaque propriétés si tu veux oui on peut faire aussi:

 bonjourJreaux62

rien n'empêche

apres perso je me pose une question

la plus part des fonctions comme color,backcolor,etc... ajoute eux même les balises font etc.....

peut être que JavaScript devrait alors mettre a jour alors sa fonction execcommand

après rien ne m'empêche l'ors de la finalisation de mon document de faire une conversion en uses css

si tu a des idées sur cette questions je suis preneur
18/08/2016, 20h54
Invité

Je parlais aussi et surtout du fait de SEPARER chaque "instruction" (clic sur icone) dans une balise individuelle.

Ce qui me semble plus facile à gérer que, comme tu sembles chercher à le faire, à regrouper plusieurs instructions dans une seule balise.
 : 2 instructions dans une même balise.
Pas évident à gérer sans prise de tête et codes complexes...

Alors que : 1 icone -> 1 balise span
C'est simple à gérer / ajouter / supprimer.
18/08/2016, 21h07
patricktoulon

re

re

oui j'avais compris mais je suis parti sur la bases du résultat que j'obtiens avec execcommand qui semble ne pas poser problème dans un control ocx (webbrowser) dans un userform

j'ai déjà fait des modules de conversion avec ma fonction BYSPAN en vba notamment dans ma contribution vba grille exel to html le raisonnement ne changeant pas je pourrais le refaire

en JavaScript

j'ai même fait une fonction stinline2outline afin de récupérer tout les Style les dé doublonné et les mettre entre balises "<style></style>"

mais je ne suis pas dans mon élément dans JavaScript je tâtonne et découvre

re solution trouvée

Bonjour a tous
bon voila j'ai bien galéré mais j'ai réussi a finir la fonction shadow

l'idée était de pouvoir revenir sur sa décision c'est a dire de changer le textshadow a volonté
en effet si je sélectionnais du texte dont une partie avait déjà un textshadow cette partie restais identique j'ai du combiner

voila la fonction

Code:

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
function shadowcolor(coul){
if (selectionne_text()){   
 var selection = selectionne_text();
 var elements=document.all;
  for (var i=0;i<elements.length;i++){  if (elements[i].innerText==selection){var elem=elements[i];var ok="ok"}}
               if(ok=="ok"){
                  var codehtml= elem.innerHTML;                                                                  
                  var colhtml=elem.all;
                                      if (colhtml.length>0){
                                           for(var i=0;i<colhtml.length;i++){
                                             var strinnerT=colhtml[i].innerHTML;
                                             var str = colhtml[i].outerHTML;
                                             var deb = "class=" + str.split("class=")[1];
                                             var fin = deb.split(">")[0];                                                                                                                                                                          
                                             var str2=  str.replace(fin,"");                                                                                                                                                                     
                                             var bal=str2.substring(0, 7);
 
                                            if (bal =="<font >"){var codehtml=codehtml.replace(str,strinnerT);}
                                            if (bal !="<font >"){ var codehtml=codehtml.replace(fin,"");}
 
                                                                                                            }
                                          }
                                                               elem.innerHTML=codehtml;
                                                       elem.style.textShadow = "0px 0px 10px "+coul; 
                                                                 }
 
if(ok !="ok"){
 var selectedText = selection.extractContents();
   var balisefont = document.createElement("FONT");
   balisefont.className="shado";
                                             balisefont.appendChild(selectedText);
                                             balisefont.style.textShadow = "0px 0px 10px "+coul; 
   selection.insertNode(balisefont);
                              }
}
}
///////////////////////////////////////////////////////////////

pour parfaire cette fonction il faudrait que j'arrive a déterminer les élément qui sont concernés par la sélection en cas de croisement (partie droite de la précédente/partie gauche de la suivante )

19/08/2016, 19h52
melka one

attention a document.all qui fonctionne uniquement avec internet explorer qui peut etre remplacé par document.getElementsByTagName("*")
19/08/2016, 20h00
patricktoulon

re

Bonjour Melka one
oui j'ai lu cela
pour ma part ca n'a pas trop d'incidence vu que le wysiwyg et dans un controls ocx(webbrowser) dans mon userform excel et vu que celui ci utilise les librairies IE09 et quelque fonctions compatible IE10 et 11 avec des balises méta bien placées

mais en tout cas je ne savais pas que l'on pouvais utiliser le joker"*" de cette manière

merci pour le tuyaux

en VB on a la fonction right("quelquechose",5)qui nous donne les 5 dernières lettres de "quelquechose"

aurait il la même chose en JavaScript ?????????

merci pour les retours
19/08/2016, 20h35
Invité

String.substr() ?

re
Bonjour Jreaux62
non ta formule donne les 5 premiers
alors j'ai trouvé toujours avec substring qui s'apparenterait a mid en VB,VBA,VBS

comme je ne connais pas la longueur de la chaine alors je fait comme ceci
Code:

1 2 3 4 5 6 7 8 9 //quand je créé la balisefont et intègre la sélection il peut arriver que j'ai plusieurs enfants dans la balisefont //il me donc tester tout les éléments avec le substring(droite) correspondant au enfant de la balisefont // j'essai de trouver la bonne formule if( balisefont.children.length >1){ var elements=editeur.getElementsByTagName("*"); for (var i=0;i<elements.length;i++){str=elements[i].innerText;alert(str.substring(str.length,str.length-4));} }
merci pour les retours c'est sympa ca m'aide a faire passer mon mal de crane :ptdr:

1 pièce(s) jointe(s)

re
bonjour a tous ca y est on y est !!!

voila maintenant c'est vraiment a volonté
en fait c'était tres simple je ne sais pas pourquoi je me suis compliqué la vie
maintenant on modifie le shadow même avec des balises entrecroisées il fallait simplement s'occuper des enfant de la balises créée après le append Child sur le selectedtext

voila maintenant plus besoins de formater avant de refaire le shadow

Code:

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
////////////////////////////////////////////////////////////////////////////////////
function shadowcolor(coul){
if (selectionne_text()){   
 var selection = selectionne_text();
 var elements=editeur.getElementsByTagName("*");
  for (var i=0;i<elements.length;i++){  if (elements[i].innerText==selection){var elem=elements[i];var ok="ok"}}
               if(ok=="ok"){
                  var codehtml= elem.innerHTML;                                                                  
                  var colhtml=elem.getElementsByTagName("*");
                                      if (colhtml.length>0){
                                           for(var i=0;i<colhtml.length;i++){
                                             var strinnerT=colhtml[i].innerHTML;
                                             var str = colhtml[i].outerHTML;
                                             var deb = "class=" + str.split("class=")[1];
                                             var fin = deb.split(">")[0];                                                                                                                                                                          
                                             var str2=  str.replace(fin,"");                                                                                                                                                                     
                                             var bal=str2.substring(0, 7);
 
                                            if (bal =="<font >"){var codehtml=codehtml.replace(str,strinnerT);}
                                            if (bal !="<font >"){ var codehtml=codehtml.replace(fin,"");}
 
                                                                                                            }
                                          }
                                                               elem.innerHTML=codehtml;
                                                       elem.style.textShadow = "0px 0px 10px "+coul; 
                                                                 }
 
if(ok !="ok"){                 //debut du if pas ok
 var selectedText = selection.extractContents();
   var balisefont = document.createElement("FONT");
   balisefont.className="shado";
                                             balisefont.appendChild(selectedText);
if (balisefont.children.length>1){for (var i=0;i<balisefont.children.length;i++){balisefont.children[i].style.textShadow = "0px 0px 10px "+coul; }}
if (balisefont.children.length<2){ balisefont.style.textShadow = "0px 0px 10px "+coul; }
selection.insertNode(balisefont);
}//fin du if pas ok 
}//fin du if selectionnetext
}//fin de fonction shado
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

merci Jreaux;Melka one;)
Pièce jointe 218309

c'est quand on croit avoir gravi les dernières marches que soit on se rend compte que l'on a oublié les clés soit on les avais dans la poche

et oui en fait j'avais très bien travaillé au debut avec mon system de if ok/if pas ok

je m'ennuyais a vouloir corriger le défaut en cas de sélection multiple balises
mais en fait pas la peine il suffit de relancer la fonction

en effet la partie 1 gère les balises entière c'est a dire que si le texte sélectionné correspond a une balise entière on applique le style et on nettoie l'intérieur innerHTML

si ca n'est pas une balise entière on créé la balise et on lui applique le textshado

mais si la sélection et multiple ca applique que sur celle de gauche

alors en relançant la fonction la sélection ayant créé la balise la première fois prend la sélection comme un entier c'est le cas d'ailleurs et donc passe par le if ok et me nettoie le code
et par la même occasion ajoute le raté de la première passe
c'est vraiment tout bête
cette fois ci c'est du perfect!!!

Code:

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
////////////////////////////////////////////////////////////////////////////////////
function shadowcolor(coul){
if (selectionne_text()){   
 var selection = selectionne_text();
 var elements=editeur.getElementsByTagName("*");
  for (var i=0;i<elements.length;i++){  if (elements[i].innerText==selection){var elem=elements[i];var ok="ok"}}
               if(ok=="ok"){
                  var codehtml= elem.innerHTML;                                                                  
                  var colhtml=elem.getElementsByTagName("*");
                                      if (colhtml.length>0){
                                           for(var i=0;i<colhtml.length;i++){
                                             var strinnerT=colhtml[i].innerHTML;
                                             var str = colhtml[i].outerHTML;
                                             var deb = "class=" + str.split("class=")[1];
                                             var fin = deb.split(">")[0];                                                                                                                                                                          
                                             var str2=  str.replace(fin,"");                                                                                                                                                                     
                                             var bal=str2.substring(0, 7);
 
                                            if (bal =="<font >"){var codehtml=codehtml.replace(str,strinnerT);}
                                            if (bal !="<font >"){ var codehtml=codehtml.replace(fin,"");}
 
                                                                                                            }
                                          }
                                                               elem.innerHTML=codehtml;
                                                       elem.style.textShadow = "0px 0px 10px "+coul; 
                                                                 }
 
if(ok !="ok"){//debut du if pas ok
 var selectedText = selection.extractContents();
   var balisefont = document.createElement("FONT");
   balisefont.className="shado";  balisefont.appendChild(selectedText);
balisefont.style.textShadow = "0px 0px 10px "+coul; 
selection.insertNode(balisefont);
// on relance la fonction la selection sera donc la balise font donc une balise entiere
//donc la fonction passera par le if ok=="ok" et netoiera le code forcement
shadowcolor(coul); 
 
}//fin du if pas ok 
}//fin du if selectionnetext
}//fin de fonction shado
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

ZUT ALORS !!!!

bon apres plein de test le raisonnement du .all sur ledocument n'était pas bon quand la sélection était un subsubsub etc... enfant

mais comme le reste du raisonnement était bon j'ai décidé de l'appliquer directement dans la balise créée pour qu'il n'y aie pas de confusion

cette fois ci ca match a tout les coups meme dans les pires structures

@Jreaux je pense que je vais me servir de ce principe pour le faire en span avec css pure j'ai tout pour éviter les doublonnage ou les balises non fermées

en tout cas voila la fonction si ca peut servir

Code:

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
////////////////////////////////////////////////////////////////////////////////////
function shadowcolor(coul){
if (selectionne_text()){   
                                        var selection = selectionne_text(); 
                                        var selectedText = selection.extractContents();
                                        var balisefont = document.createElement("FONT");
                                        balisefont.className="shado";  balisefont.appendChild(selectedText);
                                       var strbal=balisefont.innerHTML;
                                       var colhtml=balisefont.getElementsByTagName("font");
                                               if (colhtml.length>0){
                                                           for(var i=0;i<colhtml.length;i++){
                                                           var strinnerT=colhtml[i].innerHTML;
                                                           var str = colhtml[i].outerHTML;
                                                           var deb = "class=" + str.split("class=")[1];
                                                           var fin = deb.split(">")[0];                                                                                                                                                                          
                                                           var str2=  str.replace(fin,"");                                                                                                                                                                     
                                                           var bal=str2.substring(0, 7);
 
                                                           if (bal =="<font >"){var strbal=strbal.replace(str,strinnerT);}
                                                           if (bal !="<font >"){ var strbal=strbal.replace(fin,"");}
                                                           }
                                               }
                                      balisefont.innerHTML=strbal;
                                     balisefont.style.textShadow = "0px 0px 10px "+coul; 
                                     selection.insertNode(balisefont);
 
                                    }//fin du if selectionnetext
}//fin de fonction shado
//////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////////