sygad1
sygad1

Reputation: 1482

Prevent jquery draggable on parent firing child event

I have jquery draggable attached to a DIV container, each of its child elements has a click event, how do I prevent the child event from firing when I drag the main parent container.

I only want to fire the child event when I click on it, not when I drag the whole thing.

I have made a demo: http://jsfiddle.net/sygad/RgdPq/1/

Any help appreciated.

Upvotes: 2

Views: 1984

Answers (3)

Sophivorus
Sophivorus

Reputation: 3083

This issue vexed me for a couple of hours too. In the end I did the following:

var dragged = false; // Flag

$( '#parent' ).draggable({
    start: function () {
        dragged = true;
    }
});

$( '#child' ).click( function () {
    if ( dragged ) {
        dragged = true; // Reset the flag
        return;
    }
    // Your click code here
});

Upvotes: 0

Sam
Sam

Reputation: 778

I know this is an old post. I worked on something similar which vexed me for a whole day! A better solution, i.e. saving an global variable and making the objects self-contained:

$('#somethingDraggableToBeClicked').mouseup(function() {
  if ( $("#someParentOrChildElementBeingDragged").is('.ui-draggable-dragging') ) 
    return;
    //your onClick code here...
}

Upvotes: 0

j08691
j08691

Reputation: 207861

You can use a temp variable to track the drag status and then change from the click event to the mouseup event like this jsFiddle example.

jQuery

var bodyWidth = $(window).width();
var bodyHeight = $(window).height();
var dragging = false;
$("#container").draggable({
    containment: "window",
    start: function() {
        dragging = true;
        $(this).addClass('dragged')
    },
    stop: function() {
        dragging = false;
        $(this).removeClass('dragged')
    }
});

$('.box').mouseup(function() {
    if (!dragging) {
        //If there is already a new box present, remove it
        if ($('.newBox').length) {
            $('.newBox').remove();
        }

        //Get the offset of THIS box we just clicked
        var thisOffset = getOffset(this)

        //Get its left & top value
        var newBoxLeft = thisOffset.left;
        var newBoxTop = thisOffset.top;

        //Full size box will be 75% width & height of viewport
        var newBoxFinalWidth = bodyWidth * 0.75;
        var newBoxFinalHeight = bodyHeight * 0.75;

        //Full size box will be positioned center/center on the screen
        var newBoxDestLeft = (bodyWidth - newBoxFinalWidth) / 2;
        var newBoxDestTop = (bodyHeight - newBoxFinalHeight) / 2;

        //Create our new box
        var newBox = '<div class="newBox clickOut"></div>'

        //Add it to the DOM
        $('body').append(newBox);

        //Position it to overlay the box we just clicked
        $('.newBox').css({
            'left': newBoxLeft,
            'top': newBoxTop
        })

        //Animate it from the orig box position to its final width/height & left/top
        $('.newBox').delay(100).animate({
            'width': newBoxFinalWidth,
            'height': newBoxFinalHeight,
            'left': newBoxDestLeft,
            'top': newBoxDestTop
        }, 2000, 'easeOutExpo')
    }
})

//Clickout function
$(document).click(function(e) { /*if parent of this click is NOT clickout AND this click is NOT clickout then hide stuff*/
    if ($(e.target).parents().is('.clickOut') == false && $(e.target).is('.clickOut') == false) {
        clickOut();
    }
});

function clickOut() {
    $('.newBox').remove()
}

function getOffset(item) {
    var cssOffset = $(item).offset();
    var cssLeft = parseInt(cssOffset.left);
    var cssTop = parseInt(cssOffset.top);

    return {
        'left': cssLeft,
        'top': cssTop
    };
}​

Upvotes: 3

Related Questions