jQuery单击事件与弹出窗口

Question

i have following code and its not working properly,basically i am willing to add popup on my click event for confirm delete but when i click deleting the row first and then apppear popup,i am stuck and coudn't understand whats going,here is my code

$.fn.matrix.deleteCategory = function ( jqObj ) {

     jqObj.find("#award").on('click', ".categoryminus", function () {



        var CategoryClass = $(this).closest('tr').attr('class'); // table row Class 
         //split all class of current table row
        var CategoryArray = CategoryClass.split(' ');
         //delete all rows having class like C-1-o-1
        var categoryClassLike        = '[class^=' + CategoryArray[0] + '-o-]';

         //for rTenderDocument,check delete c-2,c-3 and appear popup
         if ( formType == 'rTenderDocument') {

            if ( CategoryArray[0] == 'C-2' ){

                $('#priceConfirm').bPopup();

                $('.priceConfirm').click(function(){

                    if($(this).attr('id') == 'priceDeleteNo'){
                        $('#priceConfirm').bPopup().close(); 
                        return false;

                     } else {
                        $('#priceConfirm').bPopup().close();
                        return true;

                     }
                }); 
            } else if ( CategoryArray[0] == 'C-3' ){

                $('#fixedAnnualConfirm').bPopup();

                $('.fixedAnnualConfirm').click(function(){

                    if($(this).attr('id') == 'fixedAnnualDeleteNo'){
                        $('#fixedAnnualConfirm').bPopup().close(); 
                        return false;

                     } else {
                        $('#fixedAnnualConfirm').bPopup().close();
                        return true;

                     }
                }); 
            }

         }  

         //remove all child of sub category
         $( categoryClassLike ).each(function(){

             var obj              = $(this);

             var subCategoryClass = obj.closest('tr').attr('class'); // table row Class 
             //split all class of current table row
             var subCategoryArray = subCategoryClass.split(' ');
             //delete all rows having class like C-1-o-3-So-1
             var classLike        = '[class^=' + subCategoryArray[0] + '-So-]';

            //remove all child of sub category
             $( classLike ).each(function(){
                 $(this).remove();
             });

             //remove sub category
              obj.remove();
         });

          //after removing child then delete their parent
          $(this).closest('tr').remove();

     });

     return jqObj;
   };

Answer 1

I'm not sure what your code does - but I'll give you another approach to do what you want (based on tables and popup confirmation)

Take a look at this fiddle: JSFiddle Demo

I have created a basic table with rows that contains a del button:

<table>
    <tr><td>ID</td><td>NAME</td><td>ACTION</td></tr>
    <tr>
        <td><div>1</div></td>
        <td><div>Booba</div></td>
        <td><div><button class='del'>Delete</button></div></td>
    </tr>
    <tr>
        <td><div>2</div></td>
        <td><div>Johnas</div></td>
        <td><div><button class='del'>Delete</button></div></td>
    </tr>
    <tr>
        <td><div>3</div></td>
        <td><div>Coolio</div></td>
        <td><div><button class='del'>Delete</button></div></td>
     </tr>
</table>

Added a popup container with the message,buttons and hide it:

<div class='popup' id='popup' style='display:none'>
    <div>
        <p>Please confirm the delete action</p>
        <button id='confirm'>Proceed</button>
        <button id='cancel'>Cancel</button>
    </div>
</div>

Now for the Magic part ~ as yo can see i'm using a variable to store the row I wish to delete before the popup is displayed and only when the "proceed" button is clicked this element is removed (otherwise variable is reset and popup removed):

$(function(){

//element to delete:
   var ele_del;

// Expose popup message when del button clicked:
   $('button.del').click(function(){
       event.preventDefault();
           $('#popup').fadeIn('slow');
           ele_del = $(this).closest('tr');
       return false;
    });

//delete if proceed clicked:
    $('button#confirm').click(function(){
        event.preventDefault();
            $('#popup').fadeOut('slow',function(){
               $(ele_del).find('div').each(function(){
                   $(this).slideUp('slow',function(){
                      $(ele_del).remove();
                      ele_del = "";
                   });
                });
             });
         return false;
    });

//cancel delete operation:
    $('button#cancel').click(function(){
        event.preventDefault();
            $('#popup').fadeOut('slow',function(){
                ele_del = "";
            });
        return false;
     });
});

Notes:

1. I'm using a " DIV " inside the TD's to make the slideUp effect cross-browser. Some browsers won't slideUp the TR's directly.
2. The entire script is based on the effects callback functions - If you don't want to use effects you can remove the div's and remove the TR directly.
3. If you use a dynamic table (rows are being added dynamically and not all present on DOM ready) just use the " .on('click' " instead of the .click() I used.

Have fun!