pb avec setAttribute sous IE

**pekka77** · 12/01/2006, 15h58

Bonjour,

Je sais qu'il existe une différence entre IE et FFX pour affecter une valeur à un nouvel attribut d'un objet.
Pour IE :

Code :

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

objet.onChange = function() { mon_javascript };

Pour FFX :

Code :

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

objet.setAttribute("onChange", "mon_javascript");

Voilà le code de ma page :

Code :

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

1
2
3
4
5
6
 
	  sel = document.createElement('select');
	  sel.setAttribute("id", "critere"+i);
	  sel.setAttribute("onChange", "show("+i+")");
          sel.onChange = function() { show(i); };
	  sel.setAttribute("size", "1");

show(i) étant une fct définie un peu plus haut.
Lorsque je regarde le code à l'intérieur de mon select (innerHTML) :
- Sous Firefox, je vois bien l'attribut onChange. ça marche nickel
- Sous IE, je ne vois pas l'attribut onChange !! (donc je peux pas lancer mon code javascript en changeant la valeur de mon select)

Savez vous d'ou peut venir ce mystérieux pb ?

Merci !![/code]

**SpaceFrog** · 12/01/2006, 16h00

Code :

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

sel.setAttribute("onChange", function(){show(i)});

:

**pekka77** · 12/01/2006, 16h08

Tu veux mettre ce code pour FFX ?
ça marche déjà très bien sous FFX avec mon code.

Le pb est sur IE, donc sur la ligne :

Code :

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

sel.onChange = function() { show(i); };

IE est censé comprendre cette ligne mais il en est rien....

**SpaceFrog** · 12/01/2006, 16h09

onchange ? au lieu de onChange ?

**pekka77** · 12/01/2006, 16h24

Envoyé par SpaceFrog

onchange ? au lieu de onChange ?

Nickel !!!
J'étais sur qu'il y avait une majuscule pourtant.

**pekka77** · 12/01/2006, 16h28

Bon allez comme tu as l'air de bien maitriser, je te propose un autre petit pb.
Exactement la même chose
ça marche très bien sous FFX mais pas sous IE (comme d'hab)

Code :

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

1
2
3
4
5
6
7
 
          clos = document.createElement('a');
          clos.setAttribute("href", "#");
          clos.setAttribute("onclick", "javascript:document.getElementById('all').removeChild(document.getElementById('sac"+id+"'))");
          clos.onclick = function () { document.getElementById('all').removeChild(document.getElementById('sac"+id+"')); }
          texte = document.createTextNode('Supprimer');
          clos.appendChild(texte);

L'attribut onclick n'apparait pas sous IE, sous FFX pas de pb.
Tu remarques que j'ai mis onclick et pas onClick :-)

**SpaceFrog** · 12/01/2006, 16h31

a priori et comme je le dis souvent, onclick n'attend pas une recette de crèpes suzette donc javascript: est inutile !

Code :

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

1
2
3
4
5
6
clos = document.createElement('a');
          clos.setAttribute("href", "#");
          clos.setAttribute("onclick", "document.getElementById('all').removeChild(document.getElementById('sac"+id+"'))");
          clos.onclick = function () { document.getElementById('all').removeChild(document.getElementById('sac"+id+"')); }
          texte = document.createTextNode('Supprimer');
          clos.appendChild(texte);

**pekka77** · 12/01/2006, 16h43

Oui je suis d'accord avec toi pour le "javascript:"
Mais évidemment cela ne corrige pas mon pb. As tu une solution ?
Je rappelle que la ligne qui ne focntionne pas est celle-ci :

Code :

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

clos.onclick = function () { document.getElementById('all').removeChild(document.getElementById('sac'+id)); };

Est ce le fait de mettre directement du code javascript, et pas un appel de fct comme tout à l'h, qui fait que cela ne fonctionne pas ? Perso, je ne pense pas mais c'est la seule différence que je note par rapport à l'autre pb.

Merci!

**SpaceFrog** · 12/01/2006, 16h58

ce ne serait pas all qui interfère avec le mot reservé .all du langage javascript ???

**SpaceFrog** · 12/01/2006, 17h03

si tu essaye ça :

Code :

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

clos.onclick = function () { alert('sac'+id)};

déja id est encore une fois mal choisi comme nom de variable ... confusion possible avec .id ...

**pekka77** · 12/01/2006, 18h06

Non en changeant le nom des variables ça ne marche pas mieux...

**SpaceFrog** · 12/01/2006, 18h10

et le test alert donne quoi ?

**pekka77** · 12/01/2006, 18h18

En fait ma balise de lien ne possède même pas l'attribut onclick...
Donc qd je clique desus rien ne se passe puisque le code html obtenu (que j'ai vérifié en faisant un alert du innerHTML) est :

Code :

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

<a href="#">Supprimer</a>

Tu vois que l'attribut href a été ajouté sans problème (c'est la même ligne de code pour FFX et IE)
Par contre pour ajouter un attribut évenementiel, c'est une autre histoire. Il faut les deux lignes de code que j'ai mis plus haut.
Pour FFX pas de pb ça marche
Mais pour IE, pas du tout!!

**SpaceFrog** · 12/01/2006, 18h21

tu l'append ou clos ?

**pekka77** · 12/01/2006, 18h23

??

je ne sais pas ce que ça veut dire.
clos serait un mot réservé ?
Dans mon cas, c'est le nom que j'ai donné à mon élément.

**SpaceFrog** · 12/01/2006, 18h30

non je veux dire tu fais var clos=document.createElement ....

mais je ne vois nulle part

document.body.appendChild(clos) ...

faut bien que tu l'ajoute dans le flux de la page ...

**pekka77** · 13/01/2006, 12h25

Non ce n'est pas ça le problème puisque je fais le appendChild juste à la ligne qui suit.
Je l'append à un div, qui lui est rattaché au body.
De toute manière ça se passe bien à ce niveau puisque l'objet est bien créé. Ce qui manque c'est l'attribut onclick, seulement ça!
J'ai fait des recherches et la syntaxe est la suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
Les événements
 
L'ajout d'événements, comme onclick ou onmouseover par exemple, posera également problème sous ie.
 
Sous firefox, il suffit d'indiquer : mon_element.setAttribute("onclick","code javascript");Internet explorer n'affichera pas d'erreur de code mais ne l'interprètera pas non plus.
Pour pouvoir affecter un événement sous ie, il faudra alors passer par la notation : mon_element.onclick = nomdefonction; ou alors : mon_element.onclick = function() { code javascript }
 
Tous les autres attibuts tels que id, align, lang, etc ne posent quant à eux aucun problème et s'utilisent de la même manière selon les différents navigateurs.
 
tiré de http://www.nanoum.net/blog/9_setAttribute.html

Je vous rappelle que mon code est le suivant :

Code :

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

1
2
3
4
5
6
7
8
 
clos = document.createElement('a');
clos.setAttribute("href", "#");
clos.setAttribute("onclick", "document.getElementById('all').removeChild(document.getElementById('sac"+id+"'))");
clos.onclick = function () { document.getElementById('all').removeChild(document.getElementById('sac"+id+"')); }
texte = document.createTextNode('Supprimer');
clos.appendChild(texte);
document.getElementById('sac'+ident).appendChild(clos);

Je fais donc exactement ce qui est proposé, et pourtant ça ne fct pas...

D'où peut venir le pb ?

Merci

**pedouille** · 13/01/2006, 22h12

J'ai eu exactement le même problème que toi avec l'evenement onclick. Mon évènement se déclenchait bien sur Firefox / Mozilla / Safari etc... mais sous IE, pas la moindre chose (même pas un alert('toto')

), alors que tout les autres evenements comme onmousedown, onmouseover etc... fonctionnait très bien même sur IE.

J'ai fini par me dire que l'évènement onclick n'était pas implémenté par l'interpréteur JS de IE, et que donc, fallait faire autrement. Du coup, j'ai remplacé le onclick par onmouseup (c'est pas tout a fait pareil, mais dans la plupart des cas, c'est OK), et depuis ça marche.

**siddh** · 13/01/2006, 23h40

as tu essayé un truc omme ça ?

Code :

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

1
2
3
4
5
6
7
8
 
function mafonction(){
    document.getElementById('all').removeChild(document.getElementById('sac"+id+"'));
}
....
 
clos.setAttribute("onclick", mafonction);
clos.onclick = mafonction;

bon par contre il faut que id soit déclaré de manière globale à la page