WordPress.org

Make WordPress Core

Changeset 23761


Ignore:
Timestamp:
03/20/13 05:49:29 (13 months 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.