pouzy a écrit :
<script language="javascript" type="text/javascript" src="includes/scripts/popupjs/AJS.js"></script> <script language="javascript" type="text/javascript" src="includes/scripts/popupjs/AJS_fx.js"></script> <script language="javascript" type="text/javascript" src="includes/scripts/popupjs/gb_scripts.js"></script>
|
L'attribut language, c'est de la merde.
Utiliser des majuscules dans ton url c'est un poil pourri.
pouzy a écrit :
Code :
- function montre(texte) {
- if(visibeul==false) {
- GetDaId("curseur" ).style.visibility="visible"; // Si il est cacher (la verif n'est qu'une securité) on le rend visible.
- GetDaId("curseur" ).innerHTML = texte; // Cette fonction est a améliorer, il parait qu'elle n'est pas valide (mais elle marche)
- visibeul=true;
- }
- }
- function cache() {
- if(visibeul==true) {
- GetDaId("curseur" ).style.visibility="hidden"; // Si la bulle etais visible on la cache
- visibeul=false;
- }
- }
|
|
J'aimerais bien comprendre pourquoi tu ne teste pas directement si l'élement est visible ou pas? Ta sécurité est inutile.
Code :
- GetDaId("curseur" )
|
Garder ça c'est con.
Code :
- var monCurseur = GetDaId("curseur" )
|
c'est déja moins con, sachant que la variable monCurseur restera dans le scope de la fonction
Code :
- function pikaboo(){
- var monCurseur = GetDaId("curseur" )
- monCurseur.style.display = (monCurseur.style.display !="" ) ? "" : "hidden";
- }
|
Autrement plus, simple, plus optimisé. Le mieux étant toujours d'assigner des classes css à tes objets plutot qu'un style directement via DOM.
Edit : On est peut être sauvé, tu utilises Firebug...
Clique sur l'onglet script et vérifie que ton fichier est bien chargé.
Message édité par Shinuza le 12-05-2007 à 23:20:57
---------------
Mains power can kill, and it will hurt the entire time you’re dying from it.