Discussion :

[Neitanod]

Bonjour

Voici mon probleme : j'ai deux code en javascript qui se suivent et je crois que le fait qu'il y ait deux window.onload empêche l'un des deux de marcher.

Comment faire pour que cela fonctionne?

MERCI

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
<script type="text/javascript">
function addLoadEvent(func) {
    var oldonload = window.onload;
    if (typeof window.onload != 'function') {
        window.onload = func;
    } else {
        window.onload = function() {
            if (oldonload) {
                oldonload();
            }
            func();
        }
    }
}
addLoadEvent(function() {
    slideMenu.build('sm',700,10,10,1);
});
</script>

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
<script type="text/javascript">
 
Shadowbox.loadSkin('classic', 'src/skin');
Shadowbox.loadLanguage('en', 'src/lang');
Shadowbox.loadPlayer(['flv', 'html', 'iframe', 'img', 'qt', 'swf', 'wmp'], 'src/player');
 
window.onload = function(){
    Shadowbox.init();
    initDemos();
};
 
</script>

[devyan]

Bonjour,

Il y a surtout un problème de "visibilite" de la mémorisation de onload dans le premier script :

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
<script type="text/javascript">
  function addLoadEvent(func) {
    var oldonload = window.onload; seulement accessible pendant l'exécution de addLoadEvent
    if (typeof window.onload != 'function') {
      window.onload = func;
    } else {
      window.onload = function() {
        if (oldonload) { jamais initialisée car il n'y a pas de variable globale ayant ce nom ;)
          oldonload();
        }
        func();
      }
    }
  }
  addLoadEvent(function() {
    slideMenu.build('sm',700,10,10,1);
  });
</script>

[Neitanod]

Ok...

mais quelle est la solution pour que cela marche?

[E.Bzz]

Bonjour,

Il y a surtout un problème de "visibilite" de la mémorisation de onload dans le premier script :

tu pourrais préciser ?

Tel que tu le dis, il ne me semble pas : oldonload est effectivement une variable locale, destinée à mémoriser l'état initial du onload. Mais dans la mesure où l'évènement est redéfini (surchargé) dans la même fonction, ça ne doit pas poser de problème.
A la limite, c'est plutôt le 2° script qui devrait être adapté pour utiliser lui aussi cette fonction

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<script type="text/javascript">
 
Shadowbox.loadSkin('classic', 'src/skin');
Shadowbox.loadLanguage('en', 'src/lang');
Shadowbox.loadPlayer(['flv', 'html', 'iframe', 'img', 'qt', 'swf', 'wmp'], 'src/player');
 
addLoadEvent(function() {
    Shadowbox.init();
});
 
addLoadEvent(function() {
    initDemos();
});
</script>

A+

[devyan]

Bonjour,tu pourrais préciser ?

J'ai ajouté des notes dans l'exemple de code.

Le code de l'événement fait référence à une variable locale à la fonction d'initialisation.
Donc au moment du déclenchement de l'événement cette variable n'est plus disponible (plus "visible") => if (oldonload) donnera toujours false.


complément de réponse :
Pour gérer ce genre de chose il faudrait avoir une variable globale qui sert de "pile" (c'est probablement ce qui est fait dans les JQuery pour la gestion des événements)
Je parle de pile car cela permet de mémoriser plus d'un niveau "old"


devyan

[Neitanod]

Bonjour !

Alors voici mon site internet :
VOTRE MONDE A LA CARTE/

Le premier code (voir plus haut) concerne mon menu défilant.

Le second code se trouve dans la page "Cartes et infographies" (et réalise l'effet d'ombre lorsque l'on clique sur l'un des documents).
Vous verrez d'ailleurs que j'ai du mettre un bouton "Accueil" en lieu et place de mon menu défilant car sinon l'effet d'ombre ne marche pas.

J'ai essayé le code de E.Bzz mais ça ne marche pas.

Merci

[E.Bzz]

J'ai essayé le code de E.Bzz mais ça ne marche pas.

Un peu léger comme diagnostic

Quel effet ?
Des erreurs ? Lesquelles ? Sur quelles lignes ?

A+