jquery hide() et slideDown()

$(document).ready(function(){
 $('#AddCity').hide();
 $('#AddCityA').click(function(){
     if ( AddCityVar==1 ){
         $('#AddCity').hide();
         var AddCityVar=0;
     }else{
           $('#AddCity').slideDown("slow");
           var AddCityVar=1;
     }
  });

Je ne peux pas comprendre cela, pourquoi div avec l'ID #AddCity est ouverture sur cliquez d'abord sur #AddCityA lien, mais ne va jamais cacher sur le deuxième clic? N' $('#AddCityA').cliquez sur(function() ne fonctionnent qu'une fois?

<a href="#" id="AddCityA">Add City</a> 

<div id="AddCity">
    here some code
</div>

Merci pour toute aide

OriginalL'auteur Shirker | 2013-06-23

  1. 4

    Vous semblez être re-initialisng AddCityVar chaque fois que vous affectez une valeur à elle. Vous avez besoin de l'avoir en dehors de la portée de la fonction de clic :

    $(document).ready(function(){
    $('#AddCity').hide();
    var AddCityVar= 0;
    $('#AddCityA').click(function(){
      if ( AddCityVar==1 ) {
         $('#AddCity').hide();
         AddCityVar=0;
      }
      else {
         $('#AddCity').slideDown("slow");
         AddCityVar=1;
      }
    });
 });

    Mais, comme le rôti mentionné, il neednt avoir une variable pour ce faire. Vous pouvez simplement utiliser slideToggle :

    $('#AddCityA').click(function () {
  $('#AddCity').slideToggle("slow");
});

    Une Démo : http://jsfiddle.net/hungerpain/9cdKL/

    +1 mais AddCityVar n'a pas besoin d'être dans la portée globale, btw, dans votre exemple, il n'est pas
    merci u beaucoup! oui, c'est un travail
    changé..
    ravi de vous aider . si u pense que cela a aidé u, assurez-vous de marquer la bonne réponse 🙂

    OriginalL'auteur krishgopinath

  2. 2

    Vous pouvez simplement vérifier si l'élément est visible:

    $(document).ready(function () {
    $('#AddCity').hide();
    $('#AddCityA').click(function () {
        var state = $('#AddCity').is(':visible');
        $('#AddCity')[state?'hide':'slideDown'](state ? 0 : 'slow');
    });
});

    VIOLON

    Pour une approche beaucoup plus simple, vous pouvez glisser l'élément dans les deux sens :

        $('#AddCity').hide();
    $('#AddCityA').on('click', function () {
        $('#AddCity').slideToggle('slow');
    });
    Je souhaite pouvoir voter. celui-ci est aussi bien le travail. Thank u

    OriginalL'auteur adeneo