WordPress.org
Get WordPress

Make WordPress Core

Ticket #40678: 40678.5.diff

File 40678.5.diff, 6.7 KB (added by audrasjb, 7 years ago)

Diff 4 + Add screen-reader-text hints to the top of Edit Menus tab

  • src/wp-admin/css/nav-menus.css 

    diff --git a/src/wp-admin/css/nav-menus.css b/src/wp-admin/css/nav-menus.css
index 77c6274..cd883e2 100644
    a b ul.add-menu-item-tabs li { 
    6565        position: relative;
    6666}
    6767
     68ul.nav-tab-wrapper > li {
     69        margin-bottom: 0;
     70}
     71
    6872.blank-slate .menu-settings {
    6973        border: none;
    7074        margin-top: 0;

  • src/wp-admin/includes/class-walker-nav-menu-edit.php 

    diff --git a/src/wp-admin/includes/class-walker-nav-menu-edit.php b/src/wp-admin/includes/class-walker-nav-menu-edit.php
index d53c0c8..7f5beaa 100644
    a b class Walker_Nav_Menu_Edit extends Walker_Nav_Menu { 
    116116                        <div class="menu-item-bar">
    117117                                <div class="menu-item-handle">
    118118                                        <span class="item-title"><span class="menu-item-title"><?php echo esc_html( $title ); ?></span> <span class="is-submenu" <?php echo $submenu_text; ?>><?php _e( 'sub item' ); ?></span></span>
     119                                        <span class="screen-reader-text"><?php esc_html_e( 'Press return or enter to open this section.' ); ?></span>
    119120                                        <span class="item-controls">
    120121                                                <span class="item-type"><?php echo esc_html( $item->type_label ); ?></span>
    121122                                                <span class="item-order hide-if-js">

  • src/wp-admin/includes/nav-menu.php 

    diff --git a/src/wp-admin/includes/nav-menu.php b/src/wp-admin/includes/nav-menu.php
index 5b8bfd7..547d324 100644
    a b function wp_nav_menu_item_link_meta_box() { 
    300300
    301301                <p class="button-controls wp-clearfix">
    302302                        <span class="add-to-menu">
    303                                 <input type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" value="<?php esc_attr_e( 'Add to Menu' ); ?>" name="add-custom-menu-item" id="submit-customlinkdiv" />
     303                                <button type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" name="add-custom-menu-item" id="submit-customlinkdiv">
     304                                        <?php esc_attr_e( 'Add to Menu' ); ?><span class="screen-reader-text"> <?php esc_attr_e( 'You will have to activate "save menu" button to save your changes' ); ?></span>
     305                                </button>
    304306                                <span class="spinner"></span>
    305307                        </span>
    306308                </p>
    … … function wp_nav_menu_item_post_type_meta_box( $object, $box ) { 
    631633                        </span>
    632634
    633635                        <span class="add-to-menu">
    634                                 <input type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" value="<?php esc_attr_e( 'Add to Menu' ); ?>" name="add-post-type-menu-item" id="<?php echo esc_attr( 'submit-posttype-' . $post_type_name ); ?>" />
     636                                <button type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" name="add-post-type-menu-item" id="<?php echo esc_attr( 'submit-posttype-' . $post_type_name ); ?>">
     637                                        <?php esc_attr_e( 'Add to Menu' ); ?><span class="screen-reader-text"> <?php esc_attr_e( 'You will have to activate "save menu" button to save your changes' ); ?></span>
     638                                </button>
    635639                                <span class="spinner"></span>
    636640                        </span>
    637641                </p>
    … … function wp_nav_menu_item_taxonomy_meta_box( $object, $box ) { 
    876880                        </span>
    877881
    878882                        <span class="add-to-menu">
    879                                 <input type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" value="<?php esc_attr_e( 'Add to Menu' ); ?>" name="add-taxonomy-menu-item" id="<?php echo esc_attr( 'submit-taxonomy-' . $taxonomy_name ); ?>" />
     883                                <button type="submit"<?php wp_nav_menu_disabled_check( $nav_menu_selected_id ); ?> class="button submit-add-to-menu right" name="add-taxonomy-menu-item" id="<?php echo esc_attr( 'submit-taxonomy-' . $taxonomy_name ); ?>">
     884                                        <?php esc_attr_e( 'Add to Menu' ); ?><span class="screen-reader-text"> <?php esc_attr_e( 'You will have to activate "save menu" button to save your changes' ); ?></span>
     885                                </button>
    880886                                <span class="spinner"></span>
    881887                        </span>
    882888                </p>

  • src/wp-admin/nav-menus.php 

    diff --git a/src/wp-admin/nav-menus.php b/src/wp-admin/nav-menus.php
index 7f19e86..695ae9f 100644
    a b require_once( ABSPATH . 'wp-admin/admin-header.php' ); 
    626626
    627627        <hr class="wp-header-end">
    628628
    629         <h2 class="nav-tab-wrapper wp-clearfix">
    630                 <a href="<?php echo admin_url( 'nav-menus.php' ); ?>" class="nav-tab<?php echo $nav_tab_active_class; ?>"><?php esc_html_e( 'Edit Menus' ); ?></a>
     629        <ul class="nav-tab-wrapper wp-clearfix">
     630                <li><a href="<?php echo admin_url( 'nav-menus.php' ); ?>" class="nav-tab<?php echo $nav_tab_active_class; ?>" aria-label="<?php esc_html_e( 'Main tab to edit menus' ); ?>"><?php esc_html_e( 'Edit Menus' ); ?></a></li>
    631631                <?php if ( $num_locations && $menu_count ) : ?>
    632                         <a href="<?php echo esc_url( add_query_arg( array( 'action' => 'locations' ), admin_url( 'nav-menus.php' ) ) ); ?>" class="nav-tab
     632                        <li><a href="<?php echo esc_url( add_query_arg( array( 'action' => 'locations' ), admin_url( 'nav-menus.php' ) ) ); ?>" class="nav-tab
    633633                                                                <?php
    634634                                                                if ( $locations_screen ) {
    635635                                                                        echo ' nav-tab-active';}
    636636?>
    637 "><?php esc_html_e( 'Manage Locations' ); ?></a>
     637" aria-label="<?php esc_html_e( 'Main tab to edit menus locations' ); ?>"><?php esc_html_e( 'Manage Locations' ); ?></a></li>
    638638                <?php
    639639                        endif;
    640640                ?>
    641         </h2>
     641        </ul>
    642642        <?php
    643643        foreach ( $messages as $message ) :
    644644                echo $message . "\n";
    … … require_once( ABSPATH . 'wp-admin/admin-header.php' ); 
    740740        <div class="manage-menus">
    741741                <?php if ( $menu_count < 2 ) : ?>
    742742                <span class="add-edit-menu-action">
     743                        <span class="screen-reader-text"><?php _e( 'Do not forget to activate "save menu" button to save all changes made in this screen.' ); ?></span>
    743744                        <?php
    744745                        printf(
    745746                                __( 'Edit your menu below, or <a href="%s">create a new menu</a>.' ), esc_url(
    … … if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) { 
    830831                        <input type="hidden" name="menu" id="nav-menu-meta-object-id" value="<?php echo esc_attr( $nav_menu_selected_id ); ?>" />
    831832                        <input type="hidden" name="action" value="add-menu-item" />
    832833                        <?php wp_nonce_field( 'add-menu_item', 'menu-settings-column-nonce' ); ?>
     834                        <span class="screen-reader-text"><?php _e( 'Select the items you want to display in your menu.' ); ?></span>
    833835                        <?php do_accordion_sections( 'nav-menus', 'side', null ); ?>
    834836                </form>
    835837
    … … if ( isset( $_GET['menu'] ) && '0' == $_GET['menu'] ) { 
    843845                                $new_screen_class = 'blank-slate';
    844846                        }
    845847                        ?>
     848                                <span class="screen-reader-text"><?php _e( 'Menu settings: menu location, display order of its items, labels, moving or deleting items, etc.' ); ?></span>
    846849                                <div class="menu-edit <?php echo $new_screen_class; ?>">
    847850                                        <input type="hidden" name="nav-menu-data">
    848851                                        <?php