WordPress.org

Make WordPress Core

Ticket #19592: 19592.experimental.diff

File 19592.experimental.diff, 2.3 KB (added by soulseekah, 2 years ago)

Experimental poc to show readability decrease

  • wp-admin/js/theme.dev.js

     
    1 var ThemeViewer; 
     1var ThemeViewer, ThemeScroller; 
    22 
    33(function($){ 
    44        ThemeViewer = function( args ) { 
     
    6565 * @uses theme_list_args 
    6666 * @uses $('#_ajax_fetch_list_nonce').val() 
    6767* */ 
    68 var ThemeScroller; 
    6968(function($){ 
    7069        ThemeScroller = { 
    7170                nonce: '', 
     
    165164                 * @access private 
    166165                 */ 
    167166                ajax: function() { 
    168                         var self = this; 
     167                        var self = this, query; 
    169168 
    170169                        this.querying = true; 
    171170 
    172                         var query = { 
     171                        query = { 
    173172                                action: 'fetch-list', 
    174173                                paged: this.nextPage, 
    175174                                s: theme_list_args.search, 
  • wp-admin/js/tags.dev.js

     
    3434 
    3535                $.post(ajaxurl, $('#addtag').serialize(), function(r){ 
    3636                        $('#ajax-response').empty(); 
    37                         var res = wpAjax.parseAjaxResponse(r, 'ajax-response'); 
     37                        var res, parent, term, indent, i; 
     38             
     39            res = wpAjax.parseAjaxResponse(r, 'ajax-response'); 
    3840                        if ( ! res || res.errors ) 
    3941                                return; 
    4042 
    41                         var parent = form.find('select#parent').val(); 
     43                        parent = form.find('select#parent').val(); 
    4244 
    4345                        if ( parent > 0 && $('#tag-' + parent ).length > 0 ) // If the parent exists on this page, insert it below. Else insert it at the top of the list. 
    4446                                $('.tags #tag-' + parent).after( res.responses[0].supplemental['noparents'] ); // As the parent exists, Insert the version with - - - prefixed 
     
    4850                        $('.tags .no-items').remove(); 
    4951 
    5052                        if ( form.find('select#parent') ) { 
    51                                 // Parents field exists, Add new term to the list. 
    52                                 var term = res.responses[1].supplemental; 
     53                                // Parents field exists, Add new term to the list.                 
     54                term = res.responses[1].supplemental; 
    5355 
    5456                                // Create an indent for the Parent field 
    55                                 var indent = ''; 
    56                                 for ( var i = 0; i < res.responses[1].position; i++ ) 
     57                                indent = ''; 
     58                                for ( i = 0; i < res.responses[1].position; i++ ) 
    5759                                        indent += '&nbsp;&nbsp;&nbsp;'; 
    5860 
    5961                                form.find('select#parent option:selected').after('<option value="' + term['term_id'] + '">' + indent + term['name'] + '</option>');