CODEWITHSUNDEEP
javascriptjquery
ONYX
ONYX

Reputation: 5859

document click to hide menu

My document click function isn't hiding my menu when I click the document outside of my menu. When I click the img it shows the menu and when I click the img again it hides it but when I document click I want it to hide the menu does any one know what I'm doing wrong and how to make it work.

var visible = false;
var id = $(this).attr('id');

$(document).not('#' + id + ' div:eq(1)').click(function () {
    if (visible) {            
        $('.dropdownlist .menu').hide();
        visible = false;
    }
});    


$(this).find('div:eq(1)').click(function (e) {
     var menu = $(this).parent().find('.menu');

     if (!visible) {
         menu.show();
         visible = true;
     } else if (visible) {
         menu.hide();
         visible = false;
     }
     menu.css({ 'left': $(this).position().left + $(this).width() - menu.find('ul').width(), 
                'top': $(this).position().top + $(this).height() });
 })

Upvotes: 2

Views: 2492

Answers (2)

Jasper
Jasper

Reputation: 76003

//add event.stopPropagation() when the user clicks on a .menu element
$('.menu').on('click', function (event) {

    //.stopPropagation() will stop the event from bubbling up to the document
    event.stopPropagation();
});

//add the click event handler to the image that will open/close .menu elements
$('img').on('click', function (event) {

    //we call .stopPropagation() again here so the document won't receive this event
    event.stopPropagation();

    //cache .menu element
    var $div = $('.menu');

    //this if statement determines if the .menu should be shown or hidden, in my example I'm animating its top property
    if ($div.css('top') == '-50px') {
        $div.stop().animate({top : 0}, 250);
    } else {
        $div.stop().animate({top : '-50px'}, 150);
    }
});

//add click event handler to the document to close the .menu
$(document).on('click', function () {
    $('div').stop().animate({top : '-50px'}, 150);
});

jsfiddle: http://jsfiddle.net/jasper/n5C9w/1/

Upvotes: 2

nickdos
nickdos

Reputation: 8414

I had a similar problem and solved it with the following code:

$("body").mouseup(function(){ 
    if (visible) {
        $('.dropdownlist .menu').hide();
         visible = false;
    }
});

instead of your $(document).not(.. code.

Upvotes: 2

Related Questions