JQuery click event, not working

Question

I have the following scenario.

I have a index.php page with the following JQuery code included

jQuery(document).ready(function(){
    jQuery('#sIMG img').click(function() {
        var currentSRC = jQuery(this).attr('src');
        var altSRC = jQuery(this).attr('title');
        var imgID = jQuery(this).attr('id');
        var cat = jQuery(this).attr('name');


        /*Fade, Callback, swap the alt and src, fade in */
        jQuery('#main').fadeOut('fast',function() {
            jQuery('#main').load("detail.php?id="+imgID+"&category="+cat);
            jQuery('#main').fadeIn('fast');

         });
    });
});

Now I have two div tags called #main and #right in the index.php page. When I click on a menu item right changes to a bunch of images, if I click on one of those images the above code should take effect and load into the main div, but it's just not working. the images are located within a div called sIMG. Any help will be appreciated

Are the images that are loaded in on the right (after clicking a parent image on the right) inserted via javascript or are they just hidden with css when the page loads? — Levi
– Levi, Commented Mar 17, 2010 at 12:47
Do you use a JS debugger? Check Firebug for any error messages. — Felix Kling
– Felix Kling, Commented Mar 17, 2010 at 12:48
Kindly elaborate on "just not working". Does it run at all? What happens when you stick alert all over the place? — Kobi
– Kobi, Commented Mar 17, 2010 at 12:48
If the images are dynamically loaded, you'll have to change it to: jQuery('#sIMG img').live('click', function(){ — Jim Schubert
– Jim Schubert, Commented Mar 17, 2010 at 12:51

Foreever · Accepted Answer · 2014-04-04 06:37:39Z

12

Try using live

jQuery('#sIMG img').live("click",function(){
});

As of jQuery 1.7, the .live() method is deprecated. Use .on() to attach event handlers.

jQuery('#sIMG img').on("click",function(){
});

edited Apr 4, 2014 at 6:37

Foreever

7,55410 gold badges57 silver badges59 bronze badges

answered Mar 17, 2010 at 12:49

rahul

188k50 gold badges239 silver badges266 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Loolooii Over a year ago

As off jQuery 1.7 you should use .on() instead of live().

Callat Over a year ago

In 2017 using 3.0 live is deprecated, jquery.com/upgrade-guide/3.0/…

Mike Thomsen · Accepted Answer · 2010-03-17 12:51:41Z

1

I think what you're doing is setting "click" on the array that is return there. Try this:

jQuery('#sIMG img').each(function() {
    jQuery(this).click(function() {

    });
});

answered Mar 17, 2010 at 12:51

Mike Thomsen

37.7k11 gold badges63 silver badges86 bronze badges

2 Comments

rahul Over a year ago

Is there any need to bind event handler like this to a jquery object?

Naveen Over a year ago

Why would I have to use .live in order to get this work ? Is it because i may have duplicated ids as someone mentioned in this page? I changed the ids to ensure it was unique but I still had the same problem. Using .live worked, but I want to know why this happened?

paulalexandru · Accepted Answer · 2012-11-29 08:06:23Z

0

One of the reasons that the jquery click don't work is that you have dupplicates id's in the form.

answered Nov 29, 2012 at 8:06

paulalexandru

9,5587 gold badges68 silver badges100 bronze badges

Collectives™ on Stack Overflow

JQuery click event, not working

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related