jquery: disable click function on ui dialog

Question

$(top.document).ready(function () {   

   $("*").click(processAction);



function processAction(e){
    var clicked = e.target;
    e.stopPropagation();
    alert(clicked.tagName);
    e.stopPropagation();

    switch(clicked.tagName)
    {
        case "A":
            newDialog("You've clicked on a link !", "Please Choose An Action <br> 1<br>3");         
            return false; // prevent the default action, e.g., following a link
        break;

        case "INPUT":
            newDialog("You've clicked on a form !", "Please Choose An Action <br> 1<br>3");         
            return false;
        break;

        case "IMG":
            newDialog("You've clicked on a picture !", "Please Choose An Action <br> 1<br>3");          
            return false;
        break;

        case "BUTTON":
            newDialog("You've clicked on a button !", "Please Choose An Action <br> 1<br>3");           
            return false;
        break;

        default:
            newDialog("You've clicked on a text !", "Please Choose An Action <br> 1<br>3");         
            return false;
    }
};

function newDialog(mytitle, mycontent){
   var $dialog = $('<div></div>')
        .html(mycontent)
        .dialog({
            autoOpen: false,
            open: function() {$("*").unbind('click', processAction);},
            close: function() {$("*").bind('click', processAction);},
            title: mytitle,
            buttons: { "Cancel": function() { $(this).dialog("close"); } 
                     } 
        });
    $dialog.dialog('open');

}

I tried setting open and close events to bind and unbind the click, processAction() function, however, it doesn't seem to be working as planned.

casablanca · Accepted Answer · 2010-09-23 01:36:20Z

1

Don't ever do something like $("*").click(processAction);. It's extremely inefficient plus it basically traps every click event, which is why it's causing problems with your dialog too.

A better way is to have all the content you're interested in within a certain container, and select all elements inside that container. It's still not a very good thing to do, which begs the question: what are you trying to do? Maybe there's a better way to do whatever you're currently trying to do.

answered Sep 23, 2010 at 1:36

casablanca

70.9k8 gold badges139 silver badges155 bronze badges

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

Comments

Community · Accepted Answer · 2012-08-01 07:44:44Z

0

use

e.preventDefault();

API

http://api.jquery.com/event.preventDefault/

example

function newDialog(mytitle, mycontent){
   var $dialog = $('<div id="something"></div>')
        .html(mycontent)
        .dialog({
            autoOpen: false,
            open: function() {$("*").unbind('click', processAction);},
            close: function() {$("*").bind('click', processAction);},
            title: mytitle,
            buttons: { "Cancel": function() { $(this).dialog("close"); }
                     }
        });
    $dialog.dialog('open');
}

$('#something a').live('click', function(e){ e.preventDefault(); }); # fixed spelling

edited Aug 1, 2012 at 7:44

CommunityBot

11 silver badge

answered Sep 23, 2010 at 1:29

Pramendra Gupta

14.9k4 gold badges35 silver badges35 bronze badges

Collectives™ on Stack Overflow

jquery: disable click function on ui dialog

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related