WordPress.org

Make WordPress Core

Changeset 23761


Ignore:
Timestamp:
03/20/2013 05:49:29 AM (6 years ago)
Author:
SergeyBiryukov
Message:

Use placeholders in menu strings instead of manual concatenation. fixes #14045.

Location:
trunk/wp-admin
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/wp-admin/js/nav-menu.js

    r23760 r23761  
    429429                // Where can they move this menu item?
    430430                if ( 0 !== position ) {
    431                     var thisLink = menuItem.find( '.menus-move-up' ),
    432                         thisLinkText = thisLink.text();
    433                     thisLink.prop('title', menus.move + ' ' + thisLinkText).show();
     431                    var thisLink = menuItem.find( '.menus-move-up' );
     432                    thisLink.prop( 'title', menus.moveUp ).show();
    434433                }
    435434
    436435                if ( 0 !== position && isPrimaryMenuItem ) {
    437                     var thisLink = menuItem.find( '.menus-move-top' ),
    438                         thisLinkText = thisLink.text();
    439                     thisLink.prop('title', menus.move + ' ' + thisLinkText).show();
     436                    var thisLink = menuItem.find( '.menus-move-top' );
     437                    thisLink.prop( 'title', menus.moveToTop ).show();
    440438                }
    441439
    442440                if ( position + 1 !== totalMenuItems && 0 !== position ) {
    443                     var thisLink = menuItem.find( '.menus-move-down' ),
    444                         thisLinkText = thisLink.text();
    445                     thisLink.prop('title', menus.move + ' ' + thisLinkText).show();
     441                    var thisLink = menuItem.find( '.menus-move-down' );
     442                    thisLink.prop( 'title', menus.moveDown ).show();
    446443                }
    447444
    448445                if ( 0 === position && 0 !== hasSameDepthSibling ) {
    449                     var thisLink = menuItem.find( '.menus-move-down' ),
    450                         thisLinkText = thisLink.text();
    451                     thisLink.prop('title', menus.move + ' ' + thisLinkText).show();
     446                    var thisLink = menuItem.find( '.menus-move-down' );
     447                    thisLink.prop( 'title', menus.moveDown ).show();
    452448                }
    453449
    454450                if ( ! isPrimaryMenuItem ) {
    455451                    var thisLink = menuItem.find( '.menus-move-left' ),
    456                         thisLinkText = menus.outFrom + ' ' + prevItemNameLeft;
    457                     thisLink.prop('title', menus.move + ' ' + thisLinkText).html(thisLinkText).show();
     452                        thisLinkText = menus.outFrom.replace( '%s', prevItemNameLeft );
     453                    thisLink.prop( 'title', menus.moveOutFrom.replace( '%s', prevItemNameLeft ) ).html( thisLinkText ).show();
    458454                }
    459455
     
    461457                    if ( menuItem.find( '.menu-item-data-parent-id' ).val() !== menuItem.prev().find( '.menu-item-data-db-id' ).val() ) {
    462458                        var thisLink = menuItem.find( '.menus-move-right' ),
    463                             thisLinkText = menus.under + ' ' + prevItemNameRight;
    464                         thisLink.prop('title', menus.move + ' ' + thisLinkText).html(thisLinkText).show();
     459                            thisLinkText = menus.under.replace( '%s', prevItemNameRight );
     460                        thisLink.prop( 'title', menus.moveUnder.replace( '%s', prevItemNameRight ) ).html( thisLinkText ).show();
    465461                    }
    466462                }
     
    472468
    473469                        // String together help text for primary menu items
    474                         title = itemName + '. ' + menus.menuFocus.replace('%d', itemPosition).replace('%d', totalMenuItems) + '.';
     470                        title = menus.menuFocus.replace( '%1$s', itemName ).replace( '%2$d', itemPosition ).replace( '%3$d', totalMenuItems );
    475471                } else {
    476472                    var parentItem = menuItem.prevAll( '.menu-item-depth-' + parseInt( depth - 1 ) ).first(),
     
    478474                        parentItemName = parentItem.find( '.menu-item-title' ).text(),
    479475                        subItems = $( '.menu-item .menu-item-data-parent-id[value="' + parentItemId + '"]' ),
    480                         itemPosition = $(subItems.parents('.menu-item').get().reverse()).index( menuItem ) + 1;
     476                        itemPosition = $( subItems.parents('.menu-item').get().reverse() ).index( menuItem ) + 1;
    481477
    482478                        // String together help text for sub menu items
    483 
    484                         title = itemName + '. ' + menus.subMenuFocus.replace('%d', itemPosition) + parentItemName + '.';
     479                        title = menus.subMenuFocus.replace( '%1$s', itemName ).replace( '%2$d', itemPosition ).replace( '%3$s', parentItemName );
    485480                }
    486481
  • trunk/wp-admin/nav-menus.php

    r23760 r23761  
    356356
    357357$l10n = array(
    358     "oneThemeLocationNoMenus" => $one_theme_location_no_menus,
    359     "move" => __( 'Move' ),
    360     "menuFocus" => __( 'Menu item %d of %d' ),
    361     "subMenuFocus" => __( 'Sub item number %d under' ),
    362     "under" => __( 'Under' ),
    363     "outFrom" => __( 'Out from under' )
     358    'oneThemeLocationNoMenus' => $one_theme_location_no_menus,
     359    'moveUp'       => __( 'Move up one' ),
     360    'moveDown'     => __( 'Move down one' ),
     361    'moveToTop'    => __( 'Move to the top' ),
     362    /* translators: %s: previous item name */
     363    'moveUnder'    => __( 'Move under %s' ),
     364    /* translators: %s: previous item name */
     365    'moveOutFrom'  => __( 'Move out from under %s' ),
     366    /* translators: %s: previous item name */
     367    'under'        => __( 'Under %s' ),
     368    /* translators: %s: previous item name */
     369    'outFrom'      => __( 'Out from under %s' ),
     370    /* translators: 1: item name, 2: item position, 3: total number of items */
     371    'menuFocus'    => __( '%1$s. Menu item %2$d of %3$d.' ),
     372    /* translators: 1: item name, 2: item position, 3: parent item name */
     373    'subMenuFocus' => __( '%1$s. Sub item number %2$d under %3$s.' ),
    364374);
    365375wp_localize_script( 'nav-menu', 'menus', $l10n );
Note: See TracChangeset for help on using the changeset viewer.