Discussion :

[patricktoulon]

Bonjour a tous
et oui encore moi avec mon shadow

je réécrit au propre la fonction et je rencontre une erreur qui pour moi est insensé

je prend l parent de la selection et boucle sur ces balise font
problème si il y en a pas parent est" indefined"
alors OK je met ma boucle sur les font dans un if parent!= undefined
et bien le code bloque dans le if alors qu'il ne devrait même pas y passer

Code :

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

1
2
3
4
5
6
7
8
9
10
11
function backcol(coul, MOD) {    var sel = window.getSelection();
    var parent = sel.getRangeAt(0).commonAncestorContainer;
 alert(parent.tagName);//donne undefined si il n'y a pas de balise et que du texte 
 if(parent.tagName!="undefined"){
 var f=parent.getElementsByTagName("FONT")//génère l'erreur alors qu'elle devrait Etre sauté
 for (var i = 0; i < f.length; i++) {
    if(f[i].style.backgroundColor!=""){
    f[i].setAttribute("back",f[i].style.backgroundColor);
    }
  }
}

c'est un carcan ce javascript pour moi j'avance d'un pas je recule de 3 avec des erreurs pareil

[Biboude]

Hello ! Alors déjà tu veux faire quoi avec quoi et comment ?

Si tu bloque sur une condition dans un if vérifie au préalable la valeur de ce que tu teste.

[Beginner.]

Salut,

Je ferais plutôt le teste sur le parent c'est-à-dire vérifier son nodeType :

- Si il est égale à 1 c'est que c'est un élément (balise) tu peux avoir son tagName...
- Si il est égale à 3 c'est que c'est une nodeText et donc son tagName est undefined et son nodeName est #text.

Après ce serait bien de traiter les cas où l’utilisateur voudrait appliquer un shadow alors qu'il n'a pas sélectionné de texte car dans ce cas faire var parent = sel.getRangeAt(0).commonAncestorContainer; pourrait poser problème...

[Beginner.]

Mes remarques précédentes sont toujours valables mais je viens de comprendre ce qui déconne :

Tu as écrit : if(parent.tagName!="undefined")

C'est une condition qui risque d'être toujours vraie à moins d'avoir un tagName égale à la string "undefined"... Oui la string, tu fais la comparaison avec une string alors qu'il faudrait le faire avec le type undefined comme ça : if (parent.tagName != undefined) sans les guillemets...

[NoSmoking]

Bonjour,

c'est un carcan ce javascript pour moi...

il faut dire que tu y mets de la bonne volonté !