Une fonction qui ne fonctionne pas dans les deux sens

Version imprimable

bonsoir
encore une chose que je ne vois pas ou est l'erreur
si il es invisible il s'affiche et le contraire ne fonctionne pas il ne fait rien

Code:

1
2
3
4
5
6
7
8
9
 
 (boutons[b][1] === "ajouterdiv") {
                            bout.addEventListener("click", function(event) {//event.preventDefault();
                                   //alert(event.target.outerHTML);// j'ai bien le code donc  la fonction 
                                   if(menudiv.style.visibility="hidden"){editeur.style.right="182px";menudiv.style.visibility="visible";}
                                   if(menudiv.style.visibility='visible'){editeur.style.right='0px';menudiv.style.visibility='hidden';}
 
                            }); 
}

j'ai essayé les simple et double guillemet rien ny fait:koi:

14/02/2018, 01h45
headmax

Salut je sais pas si c'est la cause de ton problème mais je pense que ça pourrait être le cas :

Lorsque tu pose ta condition tu ne vérifie pas l'égalité stricte === au contraire tu affecte menudiv.style.visibility a l'intérieur de la condition en utilisant un seul signe égal = if(menudiv.style.visibility="hidden") au lieu d'un if(menudiv.style.visibility==="hidden") se qui me parait plus juste.
14/02/2018, 02h12
psychadelic

Et il n'y a aucun message dans le debugger de ton navigateur ??

ou en avançant pas à pas dans tes fonctions ??
(touche F12 sur FireFox et les autres..., )

Citation:

Envoyé par patricktoulon

bonsoir
encore une chose que je ne vois pas ou est l'erreur
si il es invisible il s'affiche et le contraire ne fonctionne pas il ne fait rien

Code:

1
2
3
4
5
6
7
8
9
 
 (boutons[b][1] === "ajouterdiv") {
                            bout.addEventListener("click", function(event) {//event.preventDefault();
                                   //alert(event.target.outerHTML);// j'ai bien le code donc  la fonction 
                                   if(menudiv.style.visibility="hidden"){editeur.style.right="182px";menudiv.style.visibility="visible";}
                                   if(menudiv.style.visibility='visible'){editeur.style.right='0px';menudiv.style.visibility='hidden';}
 
                            }); 
}

j'ai essayé les simple et double guillemet rien ny fait:koi:

Pourquoi ne pas faire simplement ceci :

Code:

1
2
3
4
5
 
if(menudiv.style.visibility="hidden")
 {editeur.style.right="182px";menudiv.style.visibility="visible";}
 Else
  // le reste du code

Le problème ici c'est que tu n'a pas fait de else if, ducoup l'interpréteur passe dans les deux if :/

bonjour disixlis
j'avais deja essayé le if else mais c'etait pas ca l'erreur enfait c'etait tout bete et tu fait la meme le "=" c'est en fait "==="

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
if (boutons[b][1] === "ajouterdiv") {
                            bout.addEventListener("click", function(event) {event.preventDefault();
                                   //alert(event.target.outerHTML);
                                   if(menudiv.style.visibility==="hidden") {
                                       editeur.style.right="182px";menudiv.style.visibility="visible";}
                                   else {
                                   editeur.style.right="0px";menudiv.style.visibility="hidden";
                                          }
 
                            });
}

il n'y a qu'un seul truc qui me derange la premiere fois je suis obligé de cliquer 2 fois pour le faire aparraitre apres une fois comme si javascript ne capte pas le visibility:"hidden" dans le css peut etre devrais je enlever le hidden du css et le mettre en javascript a l'ouverture du document et a la creation du div car il est dynamique non ?

14/02/2018, 11h52
patricktoulon

1 pièce(s) jointe(s)

re

re
@disixlis avec un seul "=" je n'ai pas le retour il apparait mais au prochain clic il repart pas
c'est juste a la premiere utilisation du bouton apres ouverture du fichier qu'il faut cliquer 2 fois je sais pas pourquoi comme si au depart il etait visible alors que non

@psychadelic aucune erreur dans le debuger

Pièce jointe 351118
14/02/2018, 12h08
Doksuri

salut, as-tu essaye de faire un console.log() de la visibility de ton element ?

j'ai eu ce genre de probleme et me suis rendu compte que certains elements n'etaient pas pris en compte.

pour faire ce genre de tests, j'utilise le computedStyle

Code:

if(window.getComputedStyle(document.getElementById('tonElem')).display === 'none') { ... }

(je n'ai pas encore eu a le tester pour la visibility, je ne garantis rien)

1 pièce(s) jointe(s)

Bonjour Doksuri

en effet j'avais raison depuis le depart comme le visibility hidden est dans le css au depart j'ai un gros vide a l'interrogation par javascript

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
if (boutons[b][1] === "ajouterdiv") {
                            bout.addEventListener("click", function(event) {
                                   console.log("avant click " +menudiv.style.visibility);
                                   if(menudiv.style.visibility==="hidden") {
                                       editeur.style.right="182px";menudiv.style.visibility="visible";}
                                   else {
                                   editeur.style.right="0px";menudiv.style.visibility="hidden";}
                                   //event.preventDefault();
                                  console.log("apres click " + menudiv.style.visibility);
 
                            });
                    }

demo voir le resultat dans console
Pièce jointe 351122

j'ai résolu mon soucis de double clic au depart en mettant aussi la condition vide avec "||" (ou) dans le if

Code:

1
2
3
4
5
6
7
8
9
10
11
12
if (boutons[b][1] === "ajouterdiv") {
                            bout.addEventListener("click", function(event) {
                                   console.log("avant click " +menudiv.style.visibility);
                                   if(menudiv.style.visibility==="hidden"||menudiv.style.visibility==="") {
                                       editeur.style.right="182px";menudiv.style.visibility="visible";}
                                   else {
                                   editeur.style.right="0px";menudiv.style.visibility="hidden";}
                                   //event.preventDefault();
                                  console.log("apres click " + menudiv.style.visibility);
 
                            });
}

merci a tous ;)

@doksuri je supose que ta formule va chercher dans le css?

Code:

if(window.getComputedStyle(document.getElementById('tonElem')).propriété

14/02/2018, 15h58
Beginner.

Citation:

Envoyé par patricktoulon

j'ai résolu mon soucis de double clic au depart en mettant aussi la condition vide avec "||" (ou) dans le if
... if(menudiv.style.visibility==="hidden"||menudiv.style.visibility==="") {....

Ben ça alors j'ai proposé la même solution au même problème il y a quelque temps ici : #12...

Sinon la cause du problème tu l'as découverte toi-même, comme je disais dans l'autre fil "C'est normal il faut que le style CSS soit "inline"..."... Donc si on met avant le CSS inline c'est une autre solution...
14/02/2018, 17h29
patricktoulon

re

re
oui ben j'avance doucement et comme je metrise de plus en plus la console j'ai vérifié
tu a quoi comme internetexplorer toi (quelle version)?
re et non mon css n'est pas inline je fait les chose bien cette fois ci :mouarf:....quoi que tout est relatifs
14/02/2018, 18h10
Beginner.

Oui je vois bien que tu fais de plus en plus de progrès...

Sinon j'ai la version 8...
14/02/2018, 18h34
patricktoulon

re

a mince 8 c'est un peu bas il aurais fallu la 10 min
comment ca sefait tu tourne sur quoi millenium;98:mouarf: