WordPress.org

Make WordPress Core

Ticket #16793: 16793.3.diff

File 16793.3.diff, 2.3 KB (added by scribu, 3 years ago)

selector cleanup + DRY

  • wp-admin/js/inline-edit-post.dev.js

     
    88                t.what = '#post-'; 
    99 
    1010                // prepare the edit rows 
    11                 qeRow.keyup(function(e){ 
    12                         if (e.which == 27) 
    13                                 return inlineEditPost.revert(); 
    14                 }); 
    15                 bulkRow.keyup(function(e){ 
    16                         if (e.which == 27) 
    17                                 return inlineEditPost.revert(); 
    18                 }); 
     11                function revertOnEscape(e){ 
     12                        if ( e.which == 27 )    // Escape 
     13                                return inlineEditPost.revert();          
     14                } 
     15                qeRow.keyup(revertOnEscape); 
     16                bulkRow.keyup(revertOnEscape); 
    1917 
    20                 $('a.cancel', qeRow).click(function(){ 
     18                qeRow.find('a.cancel').click(function(){ 
    2119                        return inlineEditPost.revert(); 
    2220                }); 
    23                 $('a.save', qeRow).click(function(){ 
     21 
     22                qeRow.find('a.save').click(function(){ 
    2423                        return inlineEditPost.save(this); 
    2524                }); 
    26                 $('td', qeRow).keydown(function(e){ 
     25 
     26                qeRow.find('td').keydown(function(e){ 
    2727                        if ( e.which == 13 ) 
    2828                                return inlineEditPost.save(this); 
    2929                }); 
    3030 
    31                 $('a.cancel', bulkRow).click(function(){ 
     31                bulkRow.find('a.cancel').click(function(){ 
    3232                        return inlineEditPost.revert(); 
    3333                }); 
    3434 
    35                 $('#inline-edit .inline-edit-private input[value=private]').click( function(){ 
     35                qeRow.find('.inline-edit-private input[value=private]').click( function(){ 
    3636                        var pw = $('input.inline-edit-password-input'); 
    3737                        if ( $(this).attr('checked') ) { 
    3838                                pw.val('').attr('disabled', 'disabled'); 
     
    4747                        return false; 
    4848                }); 
    4949 
    50                 $('#bulk-title-div').parents('fieldset').after( 
    51                         $('#inline-edit fieldset.inline-edit-categories').clone() 
     50                bulkRow.find('.inline-edit-col-left').after( 
     51                        qeRow.find('fieldset.inline-edit-categories').clone() 
    5252                ).siblings( 'fieldset:last' ).prepend( 
    53                         $('#inline-edit label.inline-edit-tags').clone() 
     53                        qeRow.find('label.inline-edit-tags').clone() 
    5454                ); 
    5555 
    5656                // hiearchical taxonomies expandable? 
     
    6262                        $(this).hide().prev().show().parent().next().removeClass("cat-hover"); 
    6363                }); 
    6464 
    65                 $('select[name="_status"] option[value="future"]', bulkRow).remove(); 
     65                bulkRow.find('select[name="_status"] option[value="future"]').remove(); 
    6666 
    6767                $('#doaction, #doaction2').click(function(e){ 
    6868                        var n = $(this).attr('id').substr(2);