WordPress.org

Make WordPress Core

Ticket #32399: 32399.diff

File 32399.diff, 11.7 KB (added by afercia, 4 years ago)
  • src/wp-admin/includes/class-wp-comments-list-table.php

     
    246246                        $link = add_query_arg( 'comment_type', $comment_type, $link );
    247247
    248248                foreach ( $stati as $status => $label ) {
    249                         $class = ( $status === $comment_status ) ? ' class="current"' : '';
     249                        $class = $aria_current = '';
    250250
     251                        if ( $status === $comment_status ) {
     252                                $class = ' class="current"';
     253                                $aria_current = ' aria-current="page"';
     254                        }
     255
    251256                        if ( !isset( $num_comments->$status ) )
    252257                                $num_comments->$status = 10;
    253258                        $link = add_query_arg( 'comment_status', $status, $link );
     
    258263                        if ( !empty( $_REQUEST['s'] ) )
    259264                                $link = add_query_arg( 's', esc_attr( wp_unslash( $_REQUEST['s'] ) ), $link );
    260265                        */
    261                         $status_links[ $status ] = "<a href='$link'$class>" . sprintf(
     266                        $status_links[ $status ] = "<a href='$link'$class$aria_current>" . sprintf(
    262267                                translate_nooped_plural( $label, $num_comments->$status ),
    263268                                sprintf( '<span class="%s-count">%s</span>',
    264269                                        ( 'moderated' === $status ) ? 'pending' : $status,
  • src/wp-admin/includes/class-wp-ms-themes-list-table.php

     
    306306                                $url = 'themes.php';
    307307
    308308                        if ( 'search' != $type ) {
    309                                 $status_links[$type] = sprintf( "<a href='%s' %s>%s</a>",
     309                                $status_links[$type] = sprintf( "<a href='%s'%s%s>%s</a>",
    310310                                        esc_url( add_query_arg('theme_status', $type, $url) ),
    311311                                        ( $type === $status ) ? ' class="current"' : '',
     312                                        ( $type === $status ) ? ' aria-current="page"' : '',
    312313                                        sprintf( $text, number_format_i18n( $count ) )
    313314                                );
    314315                        }
  • src/wp-admin/includes/class-wp-ms-users-list-table.php

     
    135135                $total_admins = count( $super_admins );
    136136
    137137                $class = $role != 'super' ? ' class="current"' : '';
     138                $aria_current = $role != 'super' ? ' aria-current="page"' : '';
    138139                $role_links = array();
    139                 $role_links['all'] = "<a href='" . network_admin_url('users.php') . "'$class>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
     140                $role_links['all'] = "<a href='" . network_admin_url('users.php') . "'$class$aria_current>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
    140141                $class = $role === 'super' ? ' class="current"' : '';
    141                 $role_links['super'] = "<a href='" . network_admin_url('users.php?role=super') . "'$class>" . sprintf( _n( 'Super Admin <span class="count">(%s)</span>', 'Super Admins <span class="count">(%s)</span>', $total_admins ), number_format_i18n( $total_admins ) ) . '</a>';
     142                $aria_current = $role === 'super' ? ' aria-current="page"' : '';
     143                $role_links['super'] = "<a href='" . network_admin_url('users.php?role=super') . "'$class$aria_current>" . sprintf( _n( 'Super Admin <span class="count">(%s)</span>', 'Super Admins <span class="count">(%s)</span>', $total_admins ), number_format_i18n( $total_admins ) ) . '</a>';
    142144
    143145                return $role_links;
    144146        }
  • src/wp-admin/includes/class-wp-plugin-install-list-table.php

     
    286286
    287287                $display_tabs = array();
    288288                foreach ( (array) $tabs as $action => $text ) {
    289                         $class = ( $action === $tab ) ? ' current' : '';
     289                        $class = ( $action === $tab ) ? ' class="current"' : '';
     290                        $aria_current = ( $action === $tab ) ? ' aria-current="page"' : '';
    290291                        $href = self_admin_url('plugin-install.php?tab=' . $action);
    291                         $display_tabs['plugin-install-'.$action] = "<a href='$href' class='$class'>$text</a>";
     292                        $display_tabs['plugin-install-'.$action] = "<a href='$href'$class$aria_current>$text</a>";
    292293                }
    293294                // No longer a real tab.
    294295                unset( $display_tabs['plugin-install-upload'] );
  • src/wp-admin/includes/class-wp-plugins-list-table.php

     
    432432                        }
    433433
    434434                        if ( 'search' !== $type ) {
    435                                 $status_links[$type] = sprintf( "<a href='%s' %s>%s</a>",
     435                                $status_links[$type] = sprintf( "<a href='%s'%s%s>%s</a>",
    436436                                        add_query_arg('plugin_status', $type, 'plugins.php'),
    437437                                        ( $type === $status ) ? ' class="current"' : '',
     438                                        ( $type === $status ) ? ' aria-current="page"' : '',
    438439                                        sprintf( $text, number_format_i18n( $count ) )
    439440                                        );
    440441                        }
  • src/wp-admin/includes/class-wp-posts-list-table.php

     
    231231        protected function get_edit_link( $args, $label, $class = '' ) {
    232232                $url = add_query_arg( $args, 'edit.php' );
    233233
    234                 $class_html = '';
     234                $class_html = $aria_current = '';
    235235                if ( ! empty( $class ) ) {
    236236                         $class_html = sprintf(
    237237                                ' class="%s"',
    238238                                esc_attr( $class )
    239239                        );
     240                        $aria_current = ' aria-current="page"';
    240241                }
    241242
    242243                return sprintf(
    243                         '<a href="%s"%s>%s</a>',
     244                        '<a href="%s"%s%s>%s</a>',
    244245                        esc_url( $url ),
    245246                        $class_html,
     247                        $aria_current,
    246248                        $label
    247249                );
    248250        }
  • src/wp-admin/includes/class-wp-theme-install-list-table.php

     
    171171                $display_tabs = array();
    172172                foreach ( (array) $tabs as $action => $text ) {
    173173                        $class = ( $action === $tab ) ? ' class="current"' : '';
     174                        $aria_current = ( $action === $tab ) ? ' aria-current="page"' : '';
    174175                        $href = self_admin_url('theme-install.php?tab=' . $action);
    175                         $display_tabs['theme-install-'.$action] = "<a href='$href'$class>$text</a>";
     176                        $display_tabs['theme-install-'.$action] = "<a href='$href'$class$aria_current>$text</a>";
    176177                }
    177178
    178179                return $display_tabs;
  • src/wp-admin/includes/class-wp-users-list-table.php

     
    196196                        $avail_roles = array();
    197197                }
    198198
    199                 $class = empty($role) ? ' class="current"' : '';
     199                $class = $aria_current = '';
     200
     201                if ( empty( $role) ) {
     202                        $class = ' class="current"';
     203                        $aria_current = ' aria-current="page"';
     204                }
     205
    200206                $role_links = array();
    201207
    202208                if ( $count_users ) {
    203                         $role_links['all'] = "<a href='$url'$class>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
     209                        $role_links['all'] = "<a href='$url'$class$aria_current>" . sprintf( _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $total_users, 'users' ), number_format_i18n( $total_users ) ) . '</a>';
    204210                } else {
    205                         $role_links['all'] = "<a href='$url'$class>" . _x( 'All', 'users' ) . '</a>';
     211                        $role_links['all'] = "<a href='$url'$class$aria_current>" . _x( 'All', 'users' ) . '</a>';
    206212                }
    207213                foreach ( $wp_roles->get_names() as $this_role => $name ) {
    208214                        if ( $count_users && !isset($avail_roles[$this_role]) ) {
     
    209215                                continue;
    210216                        }
    211217
    212                         $class = '';
     218                        $class = $aria_current = '';
    213219
    214220                        if ( $this_role === $role ) {
    215221                                $class = ' class="current"';
     222                                $aria_current = ' aria-current="page"';
    216223                        }
    217224
    218225                        $name = translate_user_role( $name );
     
    220227                                /* translators: User role name with count */
    221228                                $name = sprintf( __('%1$s <span class="count">(%2$s)</span>'), $name, number_format_i18n( $avail_roles[$this_role] ) );
    222229                        }
    223                         $role_links[$this_role] = "<a href='" . esc_url( add_query_arg( 'role', $this_role, $url ) ) . "'$class>$name</a>";
     230                        $role_links[$this_role] = "<a href='" . esc_url( add_query_arg( 'role', $this_role, $url ) ) . "'$class$aria_current>$name</a>";
    224231                }
    225232
    226233                if ( ! $count_users || ! empty( $avail_roles['none' ] ) ) {
    227234
    228                         $class = '';
     235                        $class = $aria_current = '';
    229236
    230237                        if ( 'none' === $role ) {
    231238                                $class = ' class="current"';
     239                                $aria_current = ' aria-current="page"';
    232240                        }
    233241
    234242                        $name = __( 'No role' );
  • src/wp-admin/js/theme.js

     
    15391539                        request.tag = [ value.slice( 4 ) ];
    15401540                }
    15411541
    1542                 $( '.filter-links li > a.current' ).removeClass( 'current' );
     1542                $( '.filter-links li > a.current' )
     1543                        .removeClass( 'current' )
     1544                        .removeAttr( 'aria-current' );
     1545
    15431546                $( 'body' ).removeClass( 'show-filters filters-applied show-favorites-form' );
    15441547                $( '.drawer-toggle' ).attr( 'aria-expanded', 'false' );
    15451548
     
    16031606                this.listenTo( this.collection, 'query:fail', function() {
    16041607                        $( 'body' ).removeClass( 'loading-content' );
    16051608                        $( '.theme-browser' ).find( 'div.error' ).remove();
    1606                         $( '.theme-browser' ).find( 'div.themes' ).before( '<div class="error"><p>' + l10n.error + '</p><p><button class="button try-again">' + l10n.tryAgain + '</button></p></div>' ); 
    1607                         $( '.theme-browser .error .try-again' ).on( 'click', function( e ) { 
    1608                                 e.preventDefault(); 
    1609                                 $( 'input.wp-filter-search' ).trigger( 'input' ); 
     1609                        $( '.theme-browser' ).find( 'div.themes' ).before( '<div class="error"><p>' + l10n.error + '</p><p><button class="button try-again">' + l10n.tryAgain + '</button></p></div>' );
     1610                        $( '.theme-browser .error .try-again' ).on( 'click', function( e ) {
     1611                                e.preventDefault();
     1612                                $( 'input.wp-filter-search' ).trigger( 'input' );
    16101613                        } );
    16111614                });
    16121615
     
    16631666                // Track sorting so we can restore the correct tab when closing preview.
    16641667                themes.router.selectedTab = sort;
    16651668
    1666                 $( '.filter-links li > a, .theme-filter' ).removeClass( this.activeClass );
    1667                 $( '[data-sort="' + sort + '"]' ).addClass( this.activeClass );
     1669                $( '.filter-links li > a, .theme-filter' )
     1670                        .removeClass( this.activeClass )
     1671                        .removeAttr( 'aria-current' );
    16681672
     1673                $( '[data-sort="' + sort + '"]' )
     1674                        .addClass( this.activeClass )
     1675                        .attr( 'aria-current', 'page' );
     1676
    16691677                if ( 'favorites' === sort ) {
    16701678                        $( 'body' ).addClass( 'show-favorites-form' );
    16711679                } else {
     
    16861694                        return;
    16871695                }
    16881696
    1689                 $( '.filter-links li > a, .theme-section' ).removeClass( this.activeClass );
    1690                 $el.addClass( this.activeClass );
     1697                $( '.filter-links li > a, .theme-section' )
     1698                        .removeClass( this.activeClass )
     1699                        .removeAttr( 'aria-current' );
     1700                $el
     1701                        .addClass( this.activeClass )
     1702                        .attr( 'aria-current', 'page' );
    16911703
    16921704                if ( ! filter ) {
    16931705                        return;
     
    17251737                }
    17261738
    17271739                $( 'body' ).addClass( 'filters-applied' );
    1728                 $( '.filter-links li > a.current' ).removeClass( 'current' );
     1740                $( '.filter-links li > a.current' )
     1741                        .removeClass( 'current' )
     1742                        .removeAttr( 'aria-current' );
     1743
    17291744                filteringBy.empty();
    17301745
    17311746                _.each( tags, function( tag ) {