WordPress.org

Make WordPress Core

Changeset 32724


Ignore:
Timestamp:
06/10/2015 07:46:39 PM (4 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.