WordPress.org

Make WordPress Core

Ticket #16793: 16793.3.diff

File 16793.3.diff, 2.3 KB (added by scribu, 7 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);