WordPress.org
Get WordPress

Make WordPress Core

Changeset 19484


Ignore:
Timestamp:
11/30/2011 01:27:33 AM (13 years ago)
Author:
azaozz
Message:

Add ARIA related attributes (first take), props DrewAPicture, see #19394

Location:
trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed

  • trunk/wp-admin/menu-header.php

    r19101 r19484  
    4242        $admin_is_parent = false;
    4343        $class = array();
     44        $aria_attributes = 'tabindex="1" role="menuitem"';
    4445
    4546        if ( $first ) {
    … …  
    5455        }
    5556
    56         if ( ( $parent_file && $item[2] == $parent_file ) || ( empty($typenow) && $self == $item[2] ) )
     57        if ( ( $parent_file && $item[2] == $parent_file ) || ( empty($typenow) && $self == $item[2] ) ) {
    5758            $class[] = ! empty( $submenu_items ) ? 'wp-has-current-submenu wp-menu-open' : 'current';
    58         else
     59        } else {
    5960            $class[] = 'wp-not-current-submenu';
     61            if ( ! empty( $submenu_items ) )
     62                $aria_attributes .= ' aria-haspopup="true"';   
     63        }
    6064
    6165        if ( ! empty( $item[4] ) )
    … …  
    6367
    6468        $class = $class ? ' class="' . join( ' ', $class ) . '"' : '';
    65         $tabindex = ' tabindex="1"';
    6669        $id = ! empty( $item[5] ) ? ' id="' . preg_replace( '|[^a-zA-Z0-9_:.]|', '-', $item[5] ) . '"' : '';
    6770        $img = '';
    … …  
    8487            if ( ! empty( $menu_hook ) || ( ('index.php' != $submenu_items[0][2]) && file_exists( WP_PLUGIN_DIR . "/$menu_file" ) ) ) {
    8588                $admin_is_parent = true;
    86                 echo "<div class='wp-menu-image'><a href='admin.php?page={$submenu_items[0][2]}'>$img</a></div>$arrow<a href='admin.php?page={$submenu_items[0][2]}'$class$tabindex>$title</a>";
     89                echo "<div class='wp-menu-image'><a href='admin.php?page={$submenu_items[0][2]}'>$img</a></div>$arrow<a href='admin.php?page={$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
    8790            } else {
    88                 echo "\n\t<div class='wp-menu-image'><a href='{$submenu_items[0][2]}'>$img</a></div>$arrow<a href='{$submenu_items[0][2]}'$class$tabindex>$title</a>";
     91                echo "\n\t<div class='wp-menu-image'><a href='{$submenu_items[0][2]}'>$img</a></div>$arrow<a href='{$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
    8992            }
    9093        } elseif ( ! empty( $item[2] ) && current_user_can( $item[1] ) ) {
    … …  
    9598            if ( ! empty( $menu_hook ) || ( ('index.php' != $item[2]) && file_exists( WP_PLUGIN_DIR . "/$menu_file" ) ) ) {
    9699                $admin_is_parent = true;
    97                 echo "\n\t<div class='wp-menu-image'><a href='admin.php?page={$item[2]}'>$img</a></div>$arrow<a href='admin.php?page={$item[2]}'$class$tabindex>{$item[0]}</a>";
     100                echo "\n\t<div class='wp-menu-image'><a href='admin.php?page={$item[2]}'>$img</a></div>$arrow<a href='admin.php?page={$item[2]}'$class $aria_attributes>{$item[0]}</a>";
    98101            } else {
    99                 echo "\n\t<div class='wp-menu-image'><a href='{$item[2]}'>$img</a></div>$arrow<a href='{$item[2]}'$class$tabindex>{$item[0]}</a>";
     102                echo "\n\t<div class='wp-menu-image'><a href='{$item[2]}'>$img</a></div>$arrow<a href='{$item[2]}'$class $aria_attributes>{$item[0]}</a>";
    100103            }
    101104        }
    102105
    103106        if ( ! empty( $submenu_items ) ) {
    104             echo "\n\t<div class='wp-submenu'><div class='wp-submenu-wrap'>";
     107            echo "\n\t<div class='wp-submenu' role='menu'><div class='wp-submenu-wrap'>";
    105108            echo "<div class='wp-submenu-head'>{$item[0]}</div><ul>";
    106109            $first = true;
    … …  
    152155
    153156                    $sub_item_url = esc_url( $sub_item_url );
    154                     echo "<li$class><a href='$sub_item_url'$class$tabindex>$title</a></li>";
     157                    echo "<li$class><a href='$sub_item_url'$class $aria_attributes>$title</a></li>";
    155158                } else {
    156                     echo "<li$class><a href='{$sub_item[2]}'$class$tabindex>$title</a></li>";
     159                    echo "<li$class><a href='{$sub_item[2]}'$class $aria_attributes>$title</a></li>";
    157160                }
    158161            }
    … …  
    170173
    171174<div id="adminmenuback"></div>
    172 <div id="adminmenuwrap">
     175<div id="adminmenuwrap" role="navigation">
    173176<div id="adminmenushadow"></div>
    174 <ul id="adminmenu">
     177<ul id="adminmenu" role="menu">
    175178
    176179<?php

  • trunk/wp-includes/class-wp-admin-bar.php

    r19434 r19484  
    180180
    181181        ?>
    182         <div id="wpadminbar" class="<?php echo $class; ?>">
    183             <div class="quicklinks">
     182        <div id="wpadminbar" class="<?php echo $class; ?>" role="navigation">
     183            <div class="quicklinks" role="menubar">
    184184                <?php foreach ( $this->root->children as $group ) {
    185185                    $this->render_group( $group, 'ab-top-menu' );
    … …  
    219219                $class .= ' ' . $node->meta['class'];
    220220
    221             ?><ul id="<?php echo esc_attr( "wp-admin-bar-{$node->id}" ); ?>" class="<?php echo esc_attr( $class ); ?>"><?php
     221            ?><ul id="<?php echo esc_attr( "wp-admin-bar-{$node->id}" ); ?>" class="<?php echo esc_attr( $class ); ?>" role="menu"><?php
    222222                foreach ( $node->children as $item ) {
    223223                    $this->render_item( $item );
    … …  
    227227        // Wrap the subgroups in a div and render each individual subgroup.
    228228        elseif ( ! $is_single_group ):
    229             ?><div id="<?php echo esc_attr( "wp-admin-bar-{$node->id}-container" ); ?>" class="ab-group-container"><?php
     229            ?><div id="<?php echo esc_attr( "wp-admin-bar-{$node->id}-container" ); ?>" class="ab-group-container" role="menu"><?php
    230230                foreach ( $groups as $group ) {
    231231                    $this->render_group( $group, $class );
    … …  
    241241        $is_parent = (bool) $node->children;
    242242        $has_link  = (bool) $node->href;
    243 
    244         $menuclass = $is_parent ? 'menupop' : '';
     243        $tabindex = isset($node->meta['tabindex']) ? (int) $node->meta['tabindex'] : 10;
     244
     245        $menuclass = '';
     246        $aria_attributes = 'tabindex="' . $tabindex . '" role="menuitem"';
     247
     248        if ( $is_parent ) {
     249            $menuclass = 'menupop';
     250            $aria_attributes .= ' aria-haspopup="true"';
     251        }
     252
    245253        if ( ! empty( $node->meta['class'] ) )
    246254            $menuclass .= ' ' . $node->meta['class'];
    247255
    248         $tabindex = !empty($node->meta['tabindex']) ? $node->meta['tabindex'] : 10;
    249256        ?>
    250257
    251258        <li id="<?php echo esc_attr( "wp-admin-bar-{$node->id}" ); ?>" class="<?php echo esc_attr( $menuclass ); ?>"><?php
    252259            if ( $has_link ):
    253                 ?><a class="ab-item" tabindex="<?php echo (int) $tabindex; ?>" href="<?php echo esc_url( $node->href ) ?>"<?php
     260                ?><a class="ab-item" <?php echo $aria_attributes; ?> href="<?php echo esc_url( $node->href ) ?>"<?php
    254261                    if ( ! empty( $node->meta['onclick'] ) ) :
    255262                        ?> onclick="<?php echo esc_js( $node->meta['onclick'] ); ?>"<?php
    … …  
    263270                ?>><?php
    264271            else:
    265                 ?><div class="ab-item ab-empty-item" tabindex="<?php echo (int) $tabindex; ?>"><?php
     272                ?><div class="ab-item ab-empty-item" <?php echo $aria_attributes; ?>><?php
    266273            endif;
    267274
Note: See TracChangeset for help on using the changeset viewer.