Discussion :

[midnightweb]

Bonjour,
je dispose d'un script permettant un zoom sur un texte qu'on peut voir ici http://www.activetech.fr
Depuis la dernière mise à jour de Firefox vers Firefox 3.6.13, ce script ne fonctionne plus.
Je n'y connais rien en Javascript, comment modifier ce script pour qu'il fonctionne de nouveau ?

Code :

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

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
<script type="text/javascript">
                 i=0;
                 taille=0;
                 hauteur = (navigator.appName != "Microsoft Internet Explorer")? window.innerHeight : document.body.offsetHeight;
                 marge = hauteur/3;
                 document.getElementsByTagName('div')[0].style.marginTop = marge;
                 textanim = new Array("LE PLUS COURT CHEMIN<br/>ENTRE VOUS<br/>ET VOTRE MARCHE !");
                 function animation() {
                 document.getElementsByTagName('div')[0].style.fontSize = ""+taille+"px";
                 document.getElementsByTagName('div')[0].style.color = "#ec7523";
                 document.getElementsByTagName('div')[0].style.fontFamily = "arial";
                 document.getElementById('bienvenue').innerHTML = textanim[i];
                 if (taille < 40) {
                 taille++;
                 }
                 else {
                 i++;
                 taille=0;
                 }
                 if (i < textanim.length) {
                 setTimeout('animation()',20)
                 }
                 }
                 setTimeout('animation()',20);
                 //</script><noscript>LE PLUS COURT CHEMIN<br/>ENTRE VOUS<br/>ET VOTRE MARCHE !</noscript>

[javatwister]

je ne comprends pas le 2e appel à setTimeout;

de +, fox te signale des propriétés css inconnues;

soit dit en passant, pourquoi les 2 syntaxes

Code :

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

document.getElementsByTagName('div')[0]

et

Code :

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

document.getElementById('bienvenue')

pour désigner le même élément?

[NoSmoking]

je ne comprends pas le 2e appel à setTimeout;

le 1st setTimeout semble servir au lancement de la fonction,un appel simple est suffisant.
toutes cette partie

Code :

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

1
2
3
4
document.getElementsByTagName('div')[0].style.fontSize = "" + taille + "px";
document.getElementsByTagName('div')[0].style.color = "#ec7523";
document.getElementsByTagName('div')[0].style.fontFamily = "arial";
document.getElementById('bienvenue').innerHTML = textanim[i];

pourrait avantageusement être remplacée par

Code :

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

1
2
3
4
5
6
7
8
var oDiv = document.getElementById('bienvenue');
oDiv.style.fontSize = "" + taille + "px";
// ce qui suit n'est pas indispensable puisque constant 
// a mettre dans la feuille de style donc
oDiv.style.color = "#ec7523";
oDiv.style.fontFamily = "arial";
// pourquoi passer par un tableau puisqu'il n'a qu'un indice
oDiv.innerHTML = textanim[i];

enfin pour répondre à

Depuis la dernière mise à jour de Firefox vers Firefox 3.6.13, ce script ne fonctionne plus.

c'est surprenant effectivement, une nouvelle gestion de setTimeout????

comment modifier ce script pour qu'il fonctionne de nouveau ?

outre ce qui a déjà été dit, déplace l'écriture dans le test sur i, visiblement il y a lecture dans un indice de tableau non initialisé

Code :

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

1
2
3
4
if (i < textanim.length) {
  document.getElementById('bienvenue').innerHTML = textanim[i];
  setTimeout('animation()', 20)
}

cela reste grandement simplifiable et améliorable d'un strict point de vue de codage.

[midnightweb]

Bonjour et merci pour toutes ces réponses.

A vrai dire, n'y connaissant rien en Javascript, je vais tenter d'appliquer vos conseils, mais je n'y comprends rien