Javascript onClick“display: none;” ne fonctionne pas

J'ai une image lorsque vous cliquez sur, cette fonction s'exécute.

function img(num) {
    var src = "images/main" + num + ".jpg";
    document.getElementById("popover-image").src = src;
    $("#sheet").css("display", "block");
}

Lorsque vous cliquez sur le "X" (image), cette fonction s'exécute:

function close() {
    $("#sheet").css("display", "none");
}

mais ça ne fonctionne pas.

Ici est de savoir comment mon image est mise en place:

<img src="images/x.png" alt="Exit" onclick="close()" />

  • Comment appelez-vous cette fonction? Et pourquoi n'utilisez-vous pas .hide() et .show() à la place de ceux .css() appels? Et pourquoi ne vous mélangez la vanille DOM avec jQuery?
  • Pourquoi utiliser css si vous pouvez faire de même avec show()/hide()
  • Montrer comment l'événement est ajouté à l'X
  • "mais ça ne fonctionne pas", de la grande!
  • La fermeture fonctionne très bien pour moi. jsfiddle.net/ZMwH6
  • Merci à vous, @JohnKoerner votre méthode pour ajouter le .click() travaillé!
  • Je ne mélange, parce que je suis très nouveau à la fois JavaScript et jQuery. C'est tout ce que je sais faire.

InformationsquelleAutor gtr123 | 2013-01-12
  1. 3

    Jean Koerner code:

    $("#closeButton").click(function () {
    $("#sheet").css("display", "none");
});

    Votre gestionnaire d'événement n'était pas attaché correctement.

    InformationsquelleAutor wired_in
  2. 0

    Vous pouvez faire comme ça aussi, en utilisant le nom de la classe

    ( function($) {
  $(".btn-remove").click(function() {  
    $(this).css("display", "none");      
  });
} ) ( jQuery );
    InformationsquelleAutor chris
  3. -1

    Nous sommes tous un peu déconcerté que vous lisez comment appeler jQuery .css fonction mais pas .show(), hide(), ou attr(). Pour ce que ça vaut, voici le lien vers la fonction masquer la page. JQuery est la réponse. Je ne peux pas penser à une raison de ne pas l'utiliser partout.

    Je pense que ce que vous essayez de faire est de:

    $(document).ready( function() {
  $("img").on("click", function(num) {
     var src = "images/main" + num + ".jpg";
     $("#popover-image").attr("src", src);
     $("#sheet").attr("display", "block");
  });
  $("img[alt=Exit]").on("click", function() {
     $("#sheet").attr("display", "none");
  });
});
    • -1, display est le css et non pas un attribut. il est également sage de citer la valeur dans un attribut sélecteur
    • Merci pour le .show() et .hide(); je les ai utilisés à la place.
    InformationsquelleAutor Austin Mullins