Shadowbox et javascript

Version imprimable

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:

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:

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>

Bonjour,

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

Code:

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>

04/06/2010, 12h45
Neitanod

Ok...

mais quelle est la solution pour que cela marche?

Bonjour,

Citation:

Envoyé par devyan

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:

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+

04/06/2010, 15h54
devyan

Citation:

Envoyé par E.Bzz

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
07/06/2010, 16h19
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
07/06/2010, 16h27
E.Bzz

Citation:

Envoyé par Neitanod

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

Un peu léger comme diagnostic :aie:

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

A+
07/06/2010, 16h54
Neitanod

Oui pardon c'était un peu léger. vous l'aurez compris je suis grand débutant en javascript. Merci pour votre patience passée et future ! :D

En fait il ne s'est rien passé avec le changement (page d'essai :http://www.votremondealacarte.fr/references2.php

Peut être que ce qui m'embête se trouve plutôt dans le fichier shadowbox.js appelé pour faire marcher l'effet ombre?
http://www.votremondealacarte.fr/src/shadowbox.js