WordPress.org

Make WordPress Core

Ticket #33646: 33646.2.patch

File 33646.2.patch, 2.9 KB (added by joedolson, 6 years ago)

Fix logic error

  • wp-admin/css/common.css

     
    15241524        background-position: bottom left;
    15251525}
    15261526
    1527 #screen-options-wrap h5,
     1527#screen-options-wrap legend,
    15281528#contextual-help-wrap h5 {
    15291529        margin: 8px 0;
    15301530        font-size: 13px;
     1531        font-weight: 600;
    15311532}
    15321533
    15331534.metabox-prefs label {
  • wp-admin/includes/screen.php

     
    10881088                <div id="screen-options-wrap" class="hidden" tabindex="-1" aria-label="<?php esc_attr_e('Screen Options Tab'); ?>">
    10891089                <form id="adv-settings" method="post">
    10901090                <?php if ( isset( $wp_meta_boxes[ $this->id ] ) || $this->get_option( 'per_page' ) || ( $columns && empty( $columns['_title'] ) ) ) : ?>
    1091                         <h5><?php _e( 'Show on screen' ); ?></h5>
     1091                        <fieldset>
     1092                                <legend><?php _e( 'Show on screen' ); ?></legend>
    10921093                <?php
    10931094                endif;
    10941095
     
    11141115                                <br class="clear" />
    11151116                        </div>
    11161117                        <?php endif;
     1118                                               
    11171119                        if ( $columns ) :
    11181120                                if ( ! empty( $columns['_title'] ) ) : ?>
    1119                         <h5><?php echo $columns['_title']; ?></h5>
     1121                        <fieldset>
     1122                                <legend><?php echo $columns['_title']; ?></legend>
    11201123                        <?php endif; ?>
    11211124                        <div class="metabox-prefs">
    11221125                                <?php
     
    11391142                                ?>
    11401143                                <br class="clear" />
    11411144                        </div>
     1145                        <?php if ( ! empty( $columns['_title'] ) ) : ?>
     1146                        </fieldset>
     1147                        <?php endif; ?>                 
    11421148                <?php endif;
    1143 
     1149                if ( isset( $wp_meta_boxes[ $this->id ] ) || $this->get_option( 'per_page' ) || ( $columns && empty( $columns['_title'] ) ) ) : ?>
     1150                        </fieldset>
     1151                <?php endif;
    11441152                $this->render_screen_layout();
    11451153                $this->render_per_page_options();
    11461154                echo $this->_screen_settings;
     
    11651173                $num = $this->get_option( 'layout_columns', 'max' );
    11661174
    11671175                ?>
    1168                 <h5 class="screen-layout"><?php _e('Screen Layout'); ?></h5>
    1169                 <div class='columns-prefs'><?php
    1170                         _e('Number of Columns:');
    1171                         for ( $i = 1; $i <= $num; ++$i ):
    1172                                 ?>
    1173                                 <label class="columns-prefs-<?php echo $i; ?>">
    1174                                         <input type='radio' name='screen_columns' value='<?php echo esc_attr( $i ); ?>'
    1175                                                 <?php checked( $screen_layout_columns, $i ); ?> />
    1176                                         <?php echo esc_html( $i ); ?>
    1177                                 </label>
    1178                                 <?php
    1179                         endfor; ?>
     1176                <fieldset>
     1177                <legend class="screen-layout"><?php _e('Screen Layout'); ?></legend>
     1178                <div class='columns-prefs'>
     1179                        <span id='number-of-columns'><?php _e('Number of Columns:'); ?></span><?php
     1180                                for ( $i = 1; $i <= $num; ++$i ):
     1181                                        ?>
     1182                                        <label class="columns-prefs-<?php echo $i; ?>">
     1183                                                <input type='radio' name='screen_columns' aria-describedby='number-of-columns' value='<?php echo esc_attr( $i ); ?>'
     1184                                                        <?php checked( $screen_layout_columns, $i ); ?> />
     1185                                                <?php echo esc_html( $i ); ?>
     1186                                        </label>
     1187                                        <?php
     1188                                endfor; ?>
    11801189                </div>
     1190                </fieldset>
    11811191                <?php
    11821192        }
    11831193