Discussion Projet:Communes de France/listejavascript.js

Bon, alors la, en l'état, ce script va bouffer des ressources du browser à un niveau innaceptable. De plus du déclare deux fonctions globales alors qu'en faisant les choses bien, tu peux n'en déclarer aucune. Voici ma proposition en pseudo code:

addOnloadHook(function(){ //tu évites de déclarer un nom de fonctions
var cont;
if(!ns=0 || action!=view || !(cont=document.getElementById('carteCommuneDeFrance')) return //on quitte tout de suite si on n'est pas sur une bonne page 

cont.imgs = cont.getElementsByTagName('img')
if(cont.imgs.length!=2) return //si il y a 0,1, 3 ou + images, on ne fait rien

cont.imgs[o].style.display='none'

var anchor = document.createElement('a')
a.href="#" //si pas de page cible, pas de href. 
a.style.display = 'block' //pour que l'anchor prenne une ligne complète
a.style.textAlign = "center"
a.status = 0
a.onclick = function()
{
this.removeChild(this.firstChild)
if(this.status==0)
{
this.parentNode.imgs[O].style.display='none'
this.parentNode.imgs[1].style.display=''
this.status=1
this.appendChild(document.createTextNode("ton texte 0"))
}
else
{
this.parentNode.imgs[O].style.display=''
this.parentNode.imgs[1].style.display='none'
this.status=0
this.appendChild(document.createTextNode("ton texte 1"))
}

}

cont.insertBefore(cont.firstChild)

});

Code du modèle:

<div id='carteCommuneDeFrance'>
[[{{{1}}}]]
[[{{{2}}}]]
</div>

Le script que j'ai utilisé est une adaptation d'un des scripts déjà présent sur la page MediaWiki:Common.js. Si tu me dit que le script que tu viens de mettre au dessus fonctionne d'accord pour l'urilisation de celui-ci. Moi je t'ai mis l'exemple de ce que je voulais faire dans ta PDD. A toi de faire au mieux moi j'ai atteinds mes limites au niveau technique (j'ignore ce qu'est un browser...). A toi de finaliser le système du moment que le rendu soit proche de l'exemple que je t'ai exposé alors moi tout me va. Amicalement--Wikialine (d) 8 mai 2008 à 22:48 (CEST)

Donc, je re-dis, je n'ai pas le temps de tout faire, je veux bien t'aider et te donner des conseils pour optimiser ce script, mais je ne peux pas le développer. Dans la mesure ou, en l'état, ce script ralentirai toutes les pages de WP (je ne parle pas de la propreté du code, ni des standard du web), je m'opposerai fermement à son intégration dans common.js; donc soit tu veux bien continuer le travail, soit on en reste la. Maloq ^causer 9 mai 2008 à 01:37 (CEST)

PS: Browser.

Je viens de voir le script sur common, en effet, il est vraiment pas terrible... A sa décharge, il a un cahier des charges plus étendu. Maloq ^causer 9 mai 2008 à 01:41 (CEST)

J'ai essayer de faire fonctionner ton script en l'ajoutant dans mon monobook.js et en faisant quelques manipulations mais je n'arrive pas à le faire fonctionner. Je crois que le plus simple ce serait que je fasse une demande sur la PDD du projet javascript afin que des wikipédiens prennent le relais et finalisent les infobox. Effectivement si tu manques de temps, je comprends que ça ne sera pas évident pour toi de mettre sur pied l'infobox complète. A plusieurs programmeurs, l'infobox devrait vite être opérationnelle. En tout cas merci pour ton aide et tes conseil, c'est appréciable j'ai partager mon travail sur le projet commune de france graces à tes explication sur l'usage du monobook... amicalement--Wikialine (d) 9 mai 2008 à 11:10 (CEST)