effet zoom incompatible firefox 3.6.13

[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 :

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 :

document.getElementsByTagName('div')[0]

et

Code :

document.getElementById('bienvenue')

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

[NoSmoking]

Envoyé par javatwister

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 :

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 :

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 à

Envoyé par midnightweb

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????

Envoyé par midnightweb

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 :

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