WordPress.org
Get WordPress

Make WordPress Core

Changeset 32724


Ignore:
Timestamp:
06/10/2015 07:46:39 PM (11 years ago)
Author:
helen
Message:

List tables: Use a class instead of inline styles for hidden columns.

fixes #32608.

Location:
trunk/src/wp-admin
Files:
10 edited

Legend:

Unmodified
Added
Removed

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

    r32662 r32724  
    182182                                }
    183183
    184                                 $style = '';
    185184                                if ( in_array( $column_name, $hidden ) ) {
    186                                         $style = ' style="display:none;"';
     185                                        $classes .= ' hidden';
    187186                                }
    188187
    189                                 $attributes = "class='$classes'$style";
     188                                $attributes = "class='$classes'";
    190189
    191190                                if ( 'cb' === $column_name ) {

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

    r32722 r32724  
    947947                        $class = array( 'manage-column', "column-$column_key" );
    948948
    949                         $style = '';
    950                         if ( in_array( $column_key, $hidden ) )
    951                                 $style = 'display:none;';
    952 
    953                         $style = ' style="' . $style . '"';
     949                        if ( in_array( $column_key, $hidden ) ) {
     950                                $class[] = 'hidden';
     951                        }
    954952
    955953                        if ( 'cb' == $column_key )
    … …  
    979977                                $class = "class='" . join( ' ', $class ) . "'";
    980978
    981                         echo "<th scope='col' $id $class $style>$column_display_name</th>";
     979                        echo "<th scope='col' $id $class>$column_display_name</th>";
    982980                }
    983981        }
    … …  
    11381136                        }
    11391137
    1140                         $style = '';
    11411138                        if ( in_array( $column_name, $hidden ) ) {
    1142                                 $style = ' style="display:none;"';
     1139                                $classes .= ' hidden';
    11431140                        }
    11441141
    1145                         $attributes = "class='$classes'$style";
     1142                        $attributes = "class='$classes'";
    11461143
    11471144                        if ( 'cb' == $column_name ) {

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

    r32688 r32724  
    341341        }
    342342
    343         $style = '';
    344343        if ( in_array( $column_name, $hidden ) ) {
    345                 $style = ' style="display:none;"';
    346         }
    347 
    348         $attributes = "class='$classes'$style";
     344                $classes .= ' hidden';
     345        }
     346
     347        $attributes = "class='$classes'";
    349348
    350349        if ( 'cb' === $column_name ) {

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

    r32719 r32724  
    273273                                }
    274274
    275                                 $style = '';
    276275                                if ( in_array( $column_name, $hidden ) ) {
    277                                         $style = ' style="display:none;"';
    278                                 }
    279 
    280                                 $attributes = "class='$classes'$style";
     276                                        $classes .= ' hidden';
     277                                }
     278
     279                                $attributes = "class='$classes'";
    281280
    282281                                if ( 'cb' === $column_name ) {

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

    r32723 r32724  
    435435
    436436                foreach ( $columns as $column_name => $column_display_name ) {
    437                         $style = '';
    438                         if ( in_array( $column_name, $hidden ) )
    439                                 $style = ' style="display:none;"';
     437                        $extra_classes = '';
     438                        if ( in_array( $column_name, $hidden ) ) {
     439                                $extra_classes .= ' hidden';
     440                        }
    440441
    441442                        switch ( $column_name ) {
    … …  
    444445                                        break;
    445446                                case 'name':
    446                                         echo "<td class='theme-title column-primary'$style><strong>" . $theme->display('Name') . "</strong>";
     447                                        echo "<td class='theme-title column-primary{$extra_classes}'><strong>" . $theme->display('Name') . "</strong>";
    447448                                        echo $this->row_actions($actions, true);
    448449                                        echo "</td>";
    449450                                        break;
    450451                                case 'description':
    451                                         echo "<td class='column-description desc'$style>";
     452                                        echo "<td class='column-description desc{$extra_classes}'>";
    452453                                        if ( $theme->errors() ) {
    453454                                                $pre = $status == 'broken' ? __( 'Broken Theme:' ) . ' ' : '';
    … …  
    488489
    489490                                default:
    490                                         echo "<td class='$column_name column-$column_name'$style>";
     491                                        echo "<td class='$column_name column-$column_name{$extra_classes}'>";
    491492
    492493                                        /**

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

    r32666 r32724  
    209209                                }
    210210
    211                                 $style = '';
    212211                                if ( in_array( $column_name, $hidden ) ) {
    213                                         $style = ' style="display:none;"';
     212                                        $classes .= ' hidden';
    214213                                }
    215214
    216                                 $attributes = "class='$classes'$style";
     215                                $attributes = "class='$classes'";
    217216
    218217                                if ( 'cb' === $column_name ){

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

    r32723 r32724  
    583583
    584584                foreach ( $columns as $column_name => $column_display_name ) {
    585                         $style = '';
     585                        $extra_classes = '';
    586586                        if ( in_array( $column_name, $hidden ) ) {
    587                                 $style = ' style="display:none;"';
     587                                $extra_classes = ' hidden';
    588588                        }
    589589
    … …  
    593593                                        break;
    594594                                case 'name':
    595                                         echo "<td class='plugin-title column-primary'$style><strong>$plugin_name</strong>";
     595                                        echo "<td class='plugin-title column-primary'><strong>$plugin_name</strong>";
    596596                                        echo $this->row_actions( $actions, true );
    597597                                        echo "</td>";
    … …  
    600600                                        $classes = 'column-description desc';
    601601
    602                                         echo "<td class='$classes'$style>
     602                                        echo "<td class='$classes{$extra_classes}'>
    603603                                                <div class='plugin-description'>$description</div>
    604604                                                <div class='$class second plugin-version-author-uri'>";
    … …  
    652652                                        $classes = "$column_name column-$column_name$class";
    653653
    654                                         echo "<td class='$classes'$style>";
     654                                        echo "<td class='$classes{$extra_classes}'>";
    655655
    656656                                        /**

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

    r32721 r32724  
    723723                        }
    724724
    725                         $style = '';
    726725                        if ( in_array( $column_name, $hidden ) ) {
    727                                 $style = ' style="display:none;"';
    728                         }
    729 
    730                         $attributes = "class='$classes'$style";
     726                                $classes .= ' hidden';
     727                        }
     728
     729                        $attributes = "class='$classes'";
    731730
    732731                        if ( 'cb' === $column_name ) {

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

    r32717 r32724  
    416416                        }
    417417
    418                         $style = '';
    419418                        if ( in_array( $column_name, $hidden ) ) {
    420                                 $style = ' style="display:none;"';
    421                         }
    422 
    423                         $attributes = "class='$classes'$style";
     419                                $classes .= ' hidden';
     420                        }
     421
     422                        $attributes = "class='$classes'";
    424423
    425424                        if ( 'cb' === $column_name ) {

  • trunk/src/wp-admin/js/common.js

    r32645 r32724  
    3838
    3939        checked : function(column) {
    40                 $('.column-' + column).show();
     40                $('.column-' + column).removeClass( 'hidden' );
    4141                this.colSpanChange(+1);
    4242        },
    4343
    4444        unchecked : function(column) {
    45                 $('.column-' + column).hide();
     45                $('.column-' + column).addClass( 'hidden' );
    4646                this.colSpanChange(-1);
    4747        },
Note: See TracChangeset for help on using the changeset viewer.