Ecriture du code dans jQuery

Bonjour à tous,

Je débute avec jQuery.
J' ai fait un petit script tout simple qui permet de changer la taille d' une div lorsqu' on appuie une première fois sur une image et de revenir à sa taille initiale lorsqu' on appuie une seconde fois sur le même bouton.

Le problème est que ça ne marche que lorsqu' on appuie la première fois sur l' image mais lorsqu' on appuie une seconde fois sur l' image, la div ne revient pas à sa taille initiale (50px).

Savez-vous ce que j' ai mal écrit ,

Merci d' avance.


Voici le code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

$(function() {   //Déroule le menu $("#hamburger-mobile").click(function() {   if ($("#menu-mobile").height(50)) {   $("#menu-mobile").height(100); } else { $("#menu-mobile").height(50); }   return false;   }); });

---

Dans ton script tu lui dit si la hauteur vaut 50 alors mets une hauteur de 100 ...
Ok, mis le retour il est codé ou ??

Effectivement j' avais oublié de mettre la suite du code. Je viens de le rajouter.

Le voici :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

$(function() {   //Déroule le menu $("#hamburger-mobile").click(function() {   if ($("#menu-mobile").height(50)) {   $("#menu-mobile").height(100); } else { $("#menu-mobile").height(50); }   return false;   }); });

---

Qu' est il mal écrit ?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

$(function() {   //Déroule le menu $("#hamburger-mobile").click(function() { hh = $(this).height() if (hh == 50) { $(this).css('height', "100"); } else { $(this).css('height', "50"); } return false; }); });

---

Merci SpaceFrog.

J' ai juste précisé quelle div doit changé de hauteur parce que sinon c' était l' image auquel était assigné l' id qui changeait de hauteur.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

$(function() {   //Déroule le menu $("#hamburger-mobile").click(function() { hh = $("#menu-mobile").height() if (hh == 50) {   $("#menu-mobile").css('height', "100"); } else { $("#menu-mobile").css('height', "50"); } return false; }); });

---

Juste pour savoir on est obligé de déclarer une variable pour faire ça ?

Non, mais cela évite de relancer le sélecteur à chaque appel