Make WordPress Core

Ticket #53840: 53840.2.diff

File 53840.2.diff, 2.4 KB (added by joedolson, 3 years ago)

Update patch to remove duplicate .speak call & change text strings

  • src/js/_enqueues/lib/nav-menu.js

     
    311311                                nextItemDepth = parseInt( nextItem.menuItemDepth(), 10 ) + 1,
    312312                                prevItem = thisItem.prev(),
    313313                                prevItemDepth = parseInt( prevItem.menuItemDepth(), 10 ),
    314                                 prevItemId = prevItem.getItemData()['menu-item-db-id'];
     314                                prevItemId = prevItem.getItemData()['menu-item-db-id'],
     315                                a11ySpeech = menus[ 'moved' + dir.charAt(0).toUpperCase() + dir.slice(1) ];
    315316
    316317                        switch ( dir ) {
    317318                        case 'up':
     
    399400                        api.registerChange();
    400401                        api.refreshKeyboardAccessibility();
    401402                        api.refreshAdvancedAccessibility();
     403
     404                        if ( a11ySpeech ) {
     405                                wp.a11y.speak( a11ySpeech );
     406                        }
    402407                },
    403408
    404409                initAccessibility : function() {
     
    12241229                        $menuMarkup.hideAdvancedMenuItemFields().appendTo( api.targetList );
    12251230                        api.refreshKeyboardAccessibility();
    12261231                        api.refreshAdvancedAccessibility();
     1232                        wp.a11y.speak( menus.itemAdded );
    12271233                        $( document ).trigger( 'menu-item-added', [ $menuMarkup ] );
    12281234                },
    12291235
     
    12391245                        $menuMarkup.hideAdvancedMenuItemFields().prependTo( api.targetList );
    12401246                        api.refreshKeyboardAccessibility();
    12411247                        api.refreshAdvancedAccessibility();
     1248                        wp.a11y.speak( menus.itemAdded );
    12421249                        $( document ).trigger( 'menu-item-added', [ $menuMarkup ] );
    12431250                },
    12441251
     
    15131520                                                ins.removeClass( 'menu-instructions-inactive' );
    15141521                                        }
    15151522                                        api.refreshAdvancedAccessibility();
     1523                                        wp.a11y.speak( menus.itemRemoved );
    15161524                                });
    15171525                },
    15181526
  • src/wp-admin/nav-menus.php

     
    499499        'menuItemDeletion'        => __( 'item %s' ),
    500500        /* translators: %s: Item name. */
    501501        'itemsDeleted'            => __( 'Deleted menu item: %s.' ),
     502        'itemAdded'               => __( 'Menu item added' ),
     503        'itemRemoved'             => __( 'Menu item removed' ),
     504        'movedUp'                 => __( 'Menu item moved up' ),
     505        'movedDown'               => __( 'Menu item moved down' ),
     506        'movedTop'                => __( 'Menu item moved to the top' ),
     507        'movedLeft'               => __( 'Menu item moved out of submenu' ),
     508        'movedRight'              => __( 'Menu item is now a sub-item' ),
    502509);
    503510wp_localize_script( 'nav-menu', 'menus', $nav_menus_l10n );
    504511