Geckozone

• J'aurais aimé qu'il y ait deux petites flèches vers le bas. Pour cela j'ai mis deux fois SCspan.appendChild(SCimg); en espérant que ça insère deux fois l'image... eh bien déception, ce n'est pas le cas
  Seule la deuxième apparait. J'ai essayé de mettre une deuxième variable en me disant qu'on ne pouvait inclure chaque variable qu'une fois... non, pareil
  Qu'est-ce que j'ai faux ?
• Est-il imaginable que le "montrer les commentaires" se transforme en "masquer les commentaires" lorsque les commentaires sont affichés ? Si oui comment ?
• Je me disais qu'on pourrait peut-être stocker un cookie pour se souvenir du dernier état d'ouverture/fermeture des commentaires.
  1. Est-ce une bonne idée ?
  2. Est-ce réalisable ? (mes fonctions set cookie ont toujours été passablement foireuses à vrai dire...)
• Quelqu'un voit-il de grosses erreurs dans mon code de débutant
• Visiblement il est incompatible avec la navigation par onglet. Si on a plusieurs onglets ouverts, le script ne fonctionne que dans un... c'est mon Firefox qui est à ce point corrompu ou c'est le script ?

1. Pour phpbb j'ai reporté le bug ici mais en fait je suis en train de me dire que j'aurais du le rapporter sur le forum anglais... pas grave, de toutes façons ils n'ont pas non plus d'endroit pour les reporter... bizarre J'ai pas du le trouver
2. Je pense bien qu'il est bizarre, mais je ne vois pas comment faire autrement
3. Montrer / masquer : Ok, je vais tester ça c'est une bonne idée
4. Pour le cookie effectivement je pensais à un seul cookie général
5. Ce matin ça fonctionne. Je pense que c'était l'extension Geckomenu qui a foutu le bordel ici aussi, mais visiblement maintenant c'est mieux. Ouf (parce que c'était vriament bizarre et que je n'ai aucune envie de me refaire un profil )

en fait, le code est construit un peu bizarrement

function switchComments() {
	var monBox = document.getElementById('box_comment');
	var titreActuel=document.getElementById('commentaires').getElementsByTagName('span')[0].childNodes[1];
	
	if (monBox.className == "commentVisible") {
		monBox.className = "commentHidden";
		titreActuel.nodeValue ='\u00A0montrer les commentaires\u00A0';
	} else {
		monBox.className = "commentVisible";
		titreActuel.nodeValue ='\u00A0cacher les commentaires\u00A0';
	}
}
function switchImage(){
	// monMouseOver : variable globale
	if (monMouseOver) {
		document.down.src="/images/down_on.gif";
		document.down2.src="/images/down_on.gif";
		monMouseOver=false;
	}else{
		document.down.src="/images/down.gif";
		document.down2.src="/images/down.gif";
		monMouseOver=true;
	};
}

//window.onload = function() {
function specialCommentaire() {
	var monBox = document.getElementById('box_comment');
	var titreCommentaire = document.getElementById('commentaires');

	var SCimg = document.createElement('img');
	SCimg.setAttribute('name', 'down');
	SCimg.setAttribute('src', '/images/down.gif');
	SCimg.setAttribute('alt', 'Down');
	var SCimg2 = document.createElement('img');
	SCimg2.setAttribute('name', 'down2');
	SCimg2.setAttribute('src', '/images/down.gif');
	SCimg2.setAttribute('alt', 'Down');
	
	var SCa = document.createElement('a');
	SCa.setAttribute('href', '#commentaires');

	var SCspan = document.createElement('span');
	SCspan.appendChild(SCimg);
	SCspan.appendChild(document.createTextNode('\u00A0montrer les commentaires\u00A0'));
	SCspan.appendChild(SCimg2);
	
	SCa.appendChild(SCspan);
	
	titreCommentaire.appendChild(document.createTextNode('\u00A0'));
	titreCommentaire.appendChild(SCa);
	
	titreCommentaire.getElementsByTagName("a")[0].onclick=switchComments;
	monMouseOver=true;
	titreCommentaire.getElementsByTagName("a")[0].onmouseover=switchImage;
	titreCommentaire.getElementsByTagName("a")[0].onmouseout=switchImage;
	
	switchComments();

}
function HO_addLoadEvent(func) {
  var oldonload = window.onload;
  if (typeof window.onload != 'function') {
    window.onload = func;
  } else {
    window.onload = function() {
      oldonload();
      func();
    }
  }
}

HO_addLoadEvent(specialCommentaire);

j'ai ajouté un var devant monMouseOver=true; parce qu'il n'était pas déclaré

Avertissement : assignment to undeclared variable monMouseOver
Fichier Source : http://extensions.geckozone.org/tests/A ... omments.js
Ligne : 57

Mais de toutes façons le mouseOver ne semble pas fonctionner... une fois que la souris est passée dessus, ça clignotte, et ça arrête plus.

On ne peut pas déclarer une variable pour qu'elle soit globale ?

martin a écrit :

Mais de toutes façons le mouseOver ne semble pas fonctionner... une fois que la souris est passée dessus, ça clignotte, et ça arrête plus.

vire l'argument de la fonction switchImage(), et çà (re)fonctionnera.

martin a écrit :

On ne peut pas déclarer une variable pour qu'elle soit globale ?

Ben si, on affecte une variable sans utiliser var, et elle est globale. Javascript n'a pas "besoin" de déclaration de variable, on affecte un descripteur et c'est une variable. Et si on ajoute var, elle est locale.
Du moins à l'intérieur d'une fonction.
Tu peux déclarer ta variable en dehors d'une fonction avec var, elle sera globale à tout le script.

Mais maintenant... pourquoi les commentaires sont affichés par défaut

Désolé pour tout

KissCoolMan a écrit :Par défaut, les commentaires sont affichés, et y'a marqué "Montrer les Commentaires" (problème d'initialisation).

martin a écrit :(Re)met les parenthèses à la fonction switchComments(); dans specialCommentaire(), et la fonction sera bien appelée une première fois aprés le chargement de la page.

martin a écrit :Je sais, on peut s'y perdre, on ne met pas ces parenthèses lorsque l'on associe un évenement d'un objet à une fonction, mais là ce n'est pas le cas.

martin a écrit :Pour la déclaration de la variable globale monMouseOver, laisse là où tu l'as ma mise, mais ce n'est pas la cause de l'avertissement javascript (je n'en ai pas chez moi), celà venait des changements que tu avais fait.