WordPress.org
Get WordPress

Make WordPress Core

Ticket #26060: 26060.2.diff

File 26060.2.diff, 16.6 KB (added by helen, 10 years ago)

  • src/wp-admin/css/common.css

     
    627627        color: #000;
    628628}
    629629
    630 .alternate,
    631 .alt {
     630.striped > tbody > :nth-child(odd),
     631.alternate {
    632632        background-color: #f9f9f9;
    633633}
    634634

  • src/wp-admin/css/dashboard.css

     
    694694        margin-left: 12px;
    695695}
    696696
    697 #activity-widget #the-comment-list .alternate,
    698 #activity-widget #the-comment-list .alt {
    699         background: none;
    700 }
    701 
    702697#activity-widget #the-comment-list .comment-item {
    703698        background: #fafafa;
    704699        padding: 12px;

  • src/wp-admin/css/list-tables.css

     
    14331433
    14341434/* ms */
    14351435/* Background Color for Site Status */
    1436 .wp-list-table .site-deleted {
     1436.wp-list-table tr.site-deleted {
    14371437        background: #ff8573;
    14381438}
    1439 .wp-list-table .site-spammed {
     1439.wp-list-table tr.site-spammed {
    14401440        background: #faafaa;
    14411441}
    1442 .wp-list-table .site-archived {
     1442.wp-list-table tr.site-archived {
    14431443        background: #ffebe8;
    14441444}
    1445 .wp-list-table .site-mature {
     1445.wp-list-table tr.site-mature {
    14461446        background: #fecac2;
    14471447}
    14481448

  • src/wp-admin/import.php

     
    7676} else {
    7777        uasort( $importers, '_usort_by_first_member' );
    7878?>
    79 <table class="widefat importers">
     79<table class="widefat importers striped">
    8080
    8181<?php
    82         $alt = '';
    8382        foreach ($importers as $importer_id => $data) {
    8483                $action = '';
    8584                if ( isset( $data['install'] ) ) {
    … …  
    108107                        $action = "<a href='" . esc_url( "admin.php?import=$importer_id" ) . "' title='" . esc_attr( wptexturize( strip_tags( $data[1] ) ) ) ."'>{$data[0]}</a>";
    109108                }
    110109
    111                 $alt = $alt ? '' : ' class="alternate"';
    112110                echo "
    113                         <tr$alt>
     111                        <tr>
    114112                                <td class='import-system row-title'>$action</td>
    115113                                <td class='desc'>{$data[1]}</td>
    116114                        </tr>";

  • src/wp-admin/includes/class-wp-links-list-table.php

     
    111111        public function display_rows() {
    112112                global $cat_id;
    113113
    114                 $alt = 0;
    115 
    116114                foreach ( $this->items as $link ) {
    117115                        $link = sanitize_bookmark( $link );
    118116                        $link->link_name = esc_attr( $link->link_name );
    … …  
    122120
    123121                        $visible = ( $link->link_visible == 'Y' ) ? __( 'Yes' ) : __( 'No' );
    124122                        $rating  = $link->link_rating;
    125                         $style = ( $alt++ % 2 ) ? '' : ' class="alternate"';
    126123
    127124                        $edit_link = get_edit_bookmark_link( $link );
    128125?>
    129                 <tr id="link-<?php echo $link->link_id; ?>" <?php echo $style; ?>>
     126                <tr id="link-<?php echo $link->link_id; ?>">
    130127<?php
    131128
    132129                        list( $columns, $hidden ) = $this->get_column_info();

  • src/wp-admin/includes/class-wp-list-table.php

     
    953953         * @return array List of CSS classes for the table tag.
    954954         */
    955955        protected function get_table_classes() {
    956                 return array( 'widefat', 'fixed', $this->_args['plural'] );
     956                return array( 'widefat', 'fixed', 'striped', $this->_args['plural'] );
    957957        }
    958958
    959959        /**
    … …  
    10281028         * @param object $item The current item
    10291029         */
    10301030        public function single_row( $item ) {
    1031                 static $row_class = '';
    1032                 $row_class = ( $row_class == '' ? ' class="alternate"' : '' );
    1033 
    1034                 echo '<tr' . $row_class . '>';
     1031                echo '<tr>';
    10351032                $this->single_row_columns( $item );
    10361033                echo '</tr>';
    10371034        }

  • src/wp-admin/includes/class-wp-media-list-table.php

     
    268268                global $post;
    269269
    270270                add_filter( 'the_title','esc_html' );
    271                 $alt = '';
    272271
    273272                while ( have_posts() ) : the_post();
    274273                        $user_can_edit = current_user_can( 'edit_post', $post->ID );
    … …  
    277276                        ||  !$this->is_trash && $post->post_status == 'trash' )
    278277                                continue;
    279278
    280                         $alt = ( 'alternate' == $alt ) ? '' : 'alternate';
    281279                        $post_owner = ( get_current_user_id() == $post->post_author ) ? 'self' : 'other';
    282280                        $att_title = _draft_or_post_title();
    283281?>
    284         <tr id="post-<?php echo $post->ID; ?>" class="<?php echo trim( $alt . ' author-' . $post_owner . ' status-' . $post->post_status ); ?>">
     282        <tr id="post-<?php echo $post->ID; ?>" class="<?php echo trim( ' author-' . $post_owner . ' status-' . $post->post_status ); ?>">
    285283<?php
    286284
    287285list( $columns, $hidden ) = $this->get_column_info();

  • src/wp-admin/includes/class-wp-ms-sites-list-table.php

     
    208208                        $date = 'Y/m/d \<\b\r \/\> g:i:s a';
    209209                }
    210210
    211                 $class = '';
    212211                foreach ( $this->items as $blog ) {
    213                         $class = ( 'alternate' == $class ) ? '' : 'alternate';
     212                        $class = '';
    214213                        reset( $status_list );
    215214
    216215                        $blog_states = array();
    217216                        foreach ( $status_list as $status => $col ) {
    218217                                if ( get_blog_status( $blog['blog_id'], $status ) == 1 ) {
    219                                         $class = $col[0];
     218                                        $class = " class='{$col[0]}'";
    220219                                        $blog_states[] = $col[1];
    221220                                }
    222221                        }
    … …  
    231230                                        $blog_state .= "<span class='post-state'>$state$sep</span>";
    232231                                }
    233232                        }
    234                         echo "<tr class='$class'>";
     233                        echo "<tr{$class}>";
    235234
    236235                        $blogname = ( is_subdomain_install() ) ? str_replace( '.' . get_current_site()->domain, '', $blog['domain'] ) : $blog['path'];
    237236

  • src/wp-admin/includes/class-wp-ms-users-list-table.php

     
    149149        public function display_rows() {
    150150                global $mode;
    151151
    152                 $alt = '';
    153152                $super_admins = get_super_admins();
    154153                foreach ( $this->items as $user ) {
    155                         $alt = ( 'alternate' == $alt ) ? '' : 'alternate';
     154                        $class = '';
    156155
    157156                        $status_list = array( 'spam' => 'site-spammed', 'deleted' => 'site-deleted' );
    158157
    159158                        foreach ( $status_list as $status => $col ) {
    160159                                if ( $user->$status )
    161                                         $alt .= " $col";
     160                                        $class .= " $col";
    162161                        }
    163162
    164163                        ?>
    165                         <tr class="<?php echo $alt; ?>">
     164                        <tr class="<?php echo trim( $class ); ?>">
    166165                        <?php
    167166
    168167                        list( $columns, $hidden ) = $this->get_column_info();

  • src/wp-admin/includes/class-wp-posts-list-table.php

     
    288288        }
    289289
    290290        protected function get_table_classes() {
    291                 return array( 'widefat', 'fixed', is_post_type_hierarchical( $this->screen->post_type ) ? 'pages' : 'posts' );
     291                return array( 'widefat', 'fixed', 'striped', is_post_type_hierarchical( $this->screen->post_type ) ? 'pages' : 'posts' );
    292292        }
    293293
    294294        public function get_columns() {
    … …  
    572572
    573573        /**
    574574         * @global string $mode
    575          * @staticvar string $alternate
    576575         * @param WP_Post $post
    577576         * @param int $level
    578577         */
    579578        public function single_row( $post, $level = 0 ) {
    580579                global $mode;
    581                 static $alternate;
    582580
    583581                $global_post = get_post();
    584582                $GLOBALS['post'] = $post;
    … …  
    589587                $post_type_object = get_post_type_object( $post->post_type );
    590588                $can_edit_post = current_user_can( 'edit_post', $post->ID );
    591589
    592                 $alternate = 'alternate' == $alternate ? '' : 'alternate';
    593                 $classes = $alternate . ' iedit author-' . ( get_current_user_id() == $post->post_author ? 'self' : 'other' );
     590                $classes = 'iedit author-' . ( get_current_user_id() == $post->post_author ? 'self' : 'other' );
    594591
    595592                $lock_holder = wp_check_post_lock( $post->ID );
    596593                if ( $lock_holder ) {

  • src/wp-admin/includes/class-wp-terms-list-table.php

     
    268268
    269269        /**
    270270         * @global string $taxonomy
    271          * @staticvar string $row_class
    272271         * @param object $tag
    273272         * @param int $level
    274273         */
    … …  
    276275                global $taxonomy;
    277276                $tag = sanitize_term( $tag, $taxonomy );
    278277
    279                 static $row_class = '';
    280                 $row_class = ( $row_class == '' ? ' class="alternate"' : '' );
    281 
    282278                $this->level = $level;
    283279
    284                 echo '<tr id="tag-' . $tag->term_id . '"' . $row_class . '>';
     280                echo '<tr id="tag-' . $tag->term_id . '">';
    285281                $this->single_row_columns( $tag );
    286282                echo '</tr>';
    287283        }

  • src/wp-admin/includes/class-wp-users-list-table.php

     
    306306
    307307                $editable_roles = array_keys( get_editable_roles() );
    308308
    309                 $style = '';
    310309                foreach ( $this->items as $userid => $user_object ) {
    311310                        if ( count( $user_object->roles ) <= 1 ) {
    312311                                $role = reset( $user_object->roles );
    … …  
    319318                        if ( is_multisite() && empty( $user_object->allcaps ) )
    320319                                continue;
    321320
    322                         $style = ( ' class="alternate"' == $style ) ? '' : ' class="alternate"';
    323                         echo "\n\t" . $this->single_row( $user_object, $style, $role, isset( $post_counts ) ? $post_counts[ $userid ] : 0 );
     321                        echo "\n\t" . $this->single_row( $user_object, $style = '', $role, isset( $post_counts ) ? $post_counts[ $userid ] : 0 );
    324322                }
    325323        }
    326324
    … …  
    328326         * Generate HTML for a single row on the users.php admin panel.
    329327         *
    330328         * @since 3.1.0
     329         * @since 4.2.0 `$style` argument was deprecated.
    331330         * @access public
    332331         *
    333332         * @param object $user_object The current user object.
    334          * @param string $style       Optional. Style attributes added to the `<tr>` element.
    335          *                            Must be sanitized. Default empty.
     333         * @param string $style       Deprecated. Not used.
    336334         * @param string $role        Optional. Key for the $wp_roles array. Default empty.
    337335         * @param int    $numposts    Optional. Post count to display for this user. Defaults
    338336         *                            to zero, as in, a new user has made zero posts.
    … …  
    395393                $role_name = isset( $wp_roles->role_names[$role] ) ? translate_user_role( $wp_roles->role_names[$role] ) : __( 'None' );
    396394                $avatar = get_avatar( $user_object->ID, 32 );
    397395
    398                 $r = "<tr id='user-$user_object->ID'$style>";
     396                $r = "<tr id='user-$user_object->ID'>";
    399397
    400398                list( $columns, $hidden ) = $this->get_column_info();
    401399

  • src/wp-admin/includes/template.php

     
    589589
    590590        $r = '';
    591591        ++ $count;
    592         if ( $count % 2 )
    593                 $style = 'alternate';
    594         else
    595                 $style = '';
    596592
    597593        if ( is_serialized( $entry['meta_value'] ) ) {
    598594                if ( is_serialized_string( $entry['meta_value'] ) ) {
    … …  
    611607
    612608        $delete_nonce = wp_create_nonce( 'delete-meta_' . $entry['meta_id'] );
    613609
    614         $r .= "\n\t<tr id='meta-{$entry['meta_id']}' class='$style'>";
     610        $r .= "\n\t<tr id='meta-{$entry['meta_id']}'>";
    615611        $r .= "\n\t\t<td class='left'><label class='screen-reader-text' for='meta-{$entry['meta_id']}-key'>" . __( 'Key' ) . "</label><input name='meta[{$entry['meta_id']}][key]' id='meta-{$entry['meta_id']}-key' type='text' size='20' value='{$entry['meta_key']}' />";
    616612
    617613        $r .= "\n\t\t<div class='submit'>";

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

     
    8282                this.revert();
    8383
    8484                $('#bulk-edit td').attr('colspan', $('.widefat:first thead th:visible').length);
    85                 $('table.widefat tbody').prepend( $('#bulk-edit') );
     85                // Insert the editor at the top of the table with an empty row above to maintain zebra striping.
     86                $('table.widefat tbody').prepend( $('#bulk-edit') ).prepend('<tr class="hidden"></tr>');
    8687                $('#bulk-edit').addClass('inline-editor').show();
    8788
    8889                $( 'tbody th.check-column input[type="checkbox"]' ).each( function() {
    … …  
    128129                        fields.push('post_parent', 'page_template');
    129130                }
    130131
    131                 // add the new blank row
     132                // add the new edit row with an extra blank row underneath to maintain zebra striping.
    132133                editRow = $('#inline-edit').clone(true);
    133134                $('td', editRow).attr('colspan', $('.widefat:first thead th:visible').length);
    134135
    135                 if ( $( t.what + id ).hasClass( 'alternate' ) ) {
    136                         $(editRow).addClass('alternate');
    137                 }
    138                 $(t.what+id).hide().after(editRow);
     136                $(t.what+id).hide().before(editRow).before('<tr class="hidden"></tr>');
    139137
    140138                // populate the data
    141139                rowData = $('#inline_'+id);
    … …  
    265263
    266264                                if (r) {
    267265                                        if ( -1 !== r.indexOf( '<tr' ) ) {
    268                                                 $(inlineEditPost.what+id).remove();
     266                                                $(inlineEditPost.what+id).siblings('tr.hidden').addBack().remove();
    269267                                                $('#edit-'+id).before(r).remove();
    270268                                                $(inlineEditPost.what+id).hide().fadeIn();
    271269                                        } else {
    … …  
    275273                                } else {
    276274                                        $('#edit-'+id+' .inline-edit-save .error').html(inlineEditL10n.error).show();
    277275                                }
    278 
    279                                 if ( $('#post-'+id).prev().hasClass('alternate') ) {
    280                                         $('#post-'+id).removeClass('alternate');
    281                                 }
    282276                        },
    283277                'html');
    284278                return false;
    … …  
    291285                        $('table.widefat .spinner').hide();
    292286
    293287                        if ( 'bulk-edit' === id ) {
    294                                 $('table.widefat #bulk-edit').removeClass('inline-editor').hide();
     288                                $('table.widefat #bulk-edit').removeClass('inline-editor').hide().siblings('tr.hidden').remove();
    295289                                $('#bulk-titles').html('');
    296290                                $('#inlineedit').append( $('#bulk-edit') );
    297291                        } else {
    298                                 $('#'+id).remove();
     292                                $('#'+id).siblings('tr.hidden').addBack().remove();
    299293                                id = id.substr( id.lastIndexOf('-') + 1 );
    300294                                $(this.what+id).show();
    301295                        }

  • src/wp-admin/js/inline-edit-tax.js

     
    5656                editRow = $('#inline-edit').clone(true), rowData = $('#inline_'+id);
    5757                $('td', editRow).attr('colspan', $('.widefat:first thead th:visible').length);
    5858
    59                 if ( $( t.what + id ).hasClass( 'alternate' ) ) {
    60                         $(editRow).addClass('alternate');
    61                 }
     59                $(t.what+id).hide().before(editRow).before('<tr class="hidden"></tr>');
    6260
    63                 $(t.what+id).hide().after(editRow);
    64 
    6561                $(':input[name="name"]', editRow).val( $('.name', rowData).text() );
    6662                $(':input[name="slug"]', editRow).val( $('.slug', rowData).text() );
    6763
    … …  
    9894
    9995                                if (r) {
    10096                                        if ( -1 !== r.indexOf( '<tr' ) ) {
    101                                                 $(inlineEditTax.what+id).remove();
     97                                                $(inlineEditTax.what+id).siblings('tr.hidden').addBack().remove();
    10298                                                new_id = $(r).attr('id');
    10399
    104100                                                $('#edit-'+id).before(r).remove();
    … …  
    110106                                } else {
    111107                                        $('#edit-'+id+' .inline-edit-save .error').html(inlineEditL10n.error).show();
    112108                                }
    113 
    114                                 if ( $( row ).prev( 'tr' ).hasClass( 'alternate' ) ) {
    115                                         $(row).removeClass('alternate');
    116                                 }
    117109                        }
    118110                );
    119111                return false;
    … …  
    124116
    125117                if ( id ) {
    126118                        $('table.widefat .spinner').hide();
    127                         $('#'+id).remove();
     119                        $('#'+id).siblings('tr.hidden').addBack().remove();
    128120                        id = id.substr( id.lastIndexOf('-') + 1 );
    129121                        $(this.what+id).show();
    130122                }

  • src/wp-admin/my-sites.php

     
    7575        do_action( 'myblogs_allblogs_options' );
    7676        ?>
    7777        <br clear="all" />
    78         <table class="widefat fixed">
     78        <table class="widefat fixed striped">
    7979        <?php
    8080        /**
    8181         * Enable the Global Settings section on the My Sites screen.
    … …  
    109109                $split = $split + $cols;
    110110        }
    111111
    112         $c = '';
    113112        foreach ( $rows as $row ) {
    114                 $c = $c == 'alternate' ? '' : 'alternate';
    115                 echo "<tr class='$c'>";
     113                echo "<tr>";
    116114                $i = 0;
    117115                foreach ( $row as $user_blog ) {
    118116                        $s = $i == 3 ? '' : 'border-right: 1px solid #ccc;';