WordPress.org

Make WordPress Core


Ignore:
Timestamp:
07/24/2012 12:15:15 AM (8 years ago)
Author:
azaozz
Message:

Remove nearly all tabindex attributes from the admin, leaving them only where absolutely necessary (for now that's only the toolbar).

Add tabindex="-1" for the menu images links to avoid double tab stops there when the menu is expanded.

Fix/add auto-focus on the first input fields on the Add/Edit Post, all taxonomy, all edit taxonomy, Log In and Edit Comment screens.

See #21340.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-admin/menu-header.php

    r20698 r21311  
    4242        $admin_is_parent = false;
    4343        $class = array();
    44         $aria_attributes = 'tabindex="1"';
     44        $aria_attributes = '';
    4545
    4646        if ( $first ) {
     
    6060            $class[] = 'wp-not-current-submenu';
    6161            if ( ! empty( $submenu_items ) )
    62                 $aria_attributes .= ' aria-haspopup="true"';
     62                $aria_attributes .= 'aria-haspopup="true"';
    6363        }
    6464
     
    8888            if ( ! empty( $menu_hook ) || ( ('index.php' != $submenu_items[0][2]) && file_exists( WP_PLUGIN_DIR . "/$menu_file" ) ) ) {
    8989                $admin_is_parent = true;
    90                 echo "<div class='wp-menu-image'><a href='admin.php?page={$submenu_items[0][2]}' aria-label='$aria_label'>$img</a></div>$arrow<a href='admin.php?page={$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
     90                echo "<div class='wp-menu-image'><a href='admin.php?page={$submenu_items[0][2]}' tabindex='-1' aria-label='$aria_label'>$img</a></div>$arrow<a href='admin.php?page={$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
    9191            } else {
    92                 echo "\n\t<div class='wp-menu-image'><a href='{$submenu_items[0][2]}' aria-label='$aria_label'>$img</a></div>$arrow<a href='{$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
     92                echo "\n\t<div class='wp-menu-image'><a href='{$submenu_items[0][2]}' tabindex='-1' aria-label='$aria_label'>$img</a></div>$arrow<a href='{$submenu_items[0][2]}'$class $aria_attributes>$title</a>";
    9393            }
    9494        } elseif ( ! empty( $item[2] ) && current_user_can( $item[1] ) ) {
     
    9999            if ( ! empty( $menu_hook ) || ( ('index.php' != $item[2]) && file_exists( WP_PLUGIN_DIR . "/$menu_file" ) ) ) {
    100100                $admin_is_parent = true;
    101                 echo "\n\t<div class='wp-menu-image'><a href='admin.php?page={$item[2]}' aria-label='$aria_label'>$img</a></div>$arrow<a href='admin.php?page={$item[2]}'$class $aria_attributes>{$item[0]}</a>";
     101                echo "\n\t<div class='wp-menu-image'><a href='admin.php?page={$item[2]}' tabindex='-1' aria-label='$aria_label'>$img</a></div>$arrow<a href='admin.php?page={$item[2]}'$class $aria_attributes>{$item[0]}</a>";
    102102            } else {
    103                 echo "\n\t<div class='wp-menu-image'><a href='{$item[2]}' aria-label='$aria_label'>$img</a></div>$arrow<a href='{$item[2]}'$class $aria_attributes>{$item[0]}</a>";
     103                echo "\n\t<div class='wp-menu-image'><a href='{$item[2]}' tabindex='-1' aria-label='$aria_label'>$img</a></div>$arrow<a href='{$item[2]}'$class $aria_attributes>{$item[0]}</a>";
    104104            }
    105105        }
     
    113113                    continue;
    114114
    115                 $aria_attributes = 'tabindex="1"';
    116115                $class = array();
    117116                if ( $first ) {
     
    157156
    158157                    $sub_item_url = esc_url( $sub_item_url );
    159                     echo "<li$class><a href='$sub_item_url'$class $aria_attributes>$title</a></li>";
     158                    echo "<li$class><a href='$sub_item_url'$class>$title</a></li>";
    160159                } else {
    161                     echo "<li$class><a href='{$sub_item[2]}'$class $aria_attributes>$title</a></li>";
     160                    echo "<li$class><a href='{$sub_item[2]}'$class>$title</a></li>";
    162161                }
    163162            }
Note: See TracChangeset for help on using the changeset viewer.