I18N: Use wp.i18n for translatable strings in wp-admin/js/common.js.

This removes the usage of `wp_localize_script()` for passing translations to the script and instead adds the translatable strings in the script directly through the use of `wp.i18n` and its utilities.

Props swissspidy, ocean90.
See #20491.
Fixes #50526.
Built from https://develop.svn.wordpress.org/trunk@48267


git-svn-id: http://core.svn.wordpress.org/trunk@48036 1a063a9b-81f0-0310-95a4-ce76da25c4cd
This commit is contained in:
Dominik Schilling 2020-07-01 19:50:02 +00:00
parent d686fa266b
commit 77b2440f9f
4 changed files with 16 additions and 24 deletions

View File

@ -2,7 +2,7 @@
* @output wp-admin/js/common.js * @output wp-admin/js/common.js
*/ */
/* global setUserSetting, ajaxurl, commonL10n, alert, confirm, pagenow */ /* global setUserSetting, ajaxurl, alert, confirm, pagenow */
/* global columns, screenMeta */ /* global columns, screenMeta */
/** /**
@ -15,7 +15,8 @@
( function( $, window, undefined ) { ( function( $, window, undefined ) {
var $document = $( document ), var $document = $( document ),
$window = $( window ), $window = $( window ),
$body = $( document.body ); $body = $( document.body ),
__ = wp.i18n.__;
/** /**
* Removed in 3.3.0, needed for back-compatibility. * Removed in 3.3.0, needed for back-compatibility.
@ -188,8 +189,7 @@ window.showNotice = {
* @return {boolean} Returns true if the message is confirmed. * @return {boolean} Returns true if the message is confirmed.
*/ */
warn : function() { warn : function() {
var msg = commonL10n.warnDelete || ''; if ( confirm( __( 'You are about to permanently delete these items from your site.\nThis action cannot be undone.\n\'Cancel\' to stop, \'OK\' to delete.' ) ) ) {
if ( confirm(msg) ) {
return true; return true;
} }
@ -757,11 +757,10 @@ $document.ready( function() {
function makeNoticesDismissible() { function makeNoticesDismissible() {
$( '.notice.is-dismissible' ).each( function() { $( '.notice.is-dismissible' ).each( function() {
var $el = $( this ), var $el = $( this ),
$button = $( '<button type="button" class="notice-dismiss"><span class="screen-reader-text"></span></button>' ), $button = $( '<button type="button" class="notice-dismiss"><span class="screen-reader-text"></span></button>' );
btnText = commonL10n.dismiss || '';
// Ensure plain text. // Ensure plain text.
$button.find( '.screen-reader-text' ).text( btnText ); $button.find( '.screen-reader-text' ).text( __( 'Dismiss this notice.' ) );
$button.on( 'click.wp-dismiss-notice', function( event ) { $button.on( 'click.wp-dismiss-notice', function( event ) {
event.preventDefault(); event.preventDefault();
$el.fadeTo( 100, 0, function() { $el.fadeTo( 100, 0, function() {
@ -1559,12 +1558,14 @@ $document.ready( function() {
*/ */
$document.on( 'wp-menu-state-set wp-collapse-menu', function( event, eventData ) { $document.on( 'wp-menu-state-set wp-collapse-menu', function( event, eventData ) {
var $collapseButton = $( '#collapse-button' ), var $collapseButton = $( '#collapse-button' ),
ariaExpanded = 'true', ariaExpanded, ariaLabelText;
ariaLabelText = commonL10n.collapseMenu;
if ( 'folded' === eventData.state ) { if ( 'folded' === eventData.state ) {
ariaExpanded = 'false'; ariaExpanded = 'false';
ariaLabelText = commonL10n.expandMenu; ariaLabelText = __( 'Expand Main menu' );
} else {
ariaExpanded = 'true';
ariaLabelText = __( 'Collapse Main menu' );
} }
$collapseButton.attr({ $collapseButton.attr({

File diff suppressed because one or more lines are too long

View File

@ -608,16 +608,7 @@ function wp_default_scripts( $scripts ) {
); );
$scripts->add( 'common', "/wp-admin/js/common$suffix.js", array( 'jquery', 'hoverIntent', 'utils' ), false, 1 ); $scripts->add( 'common', "/wp-admin/js/common$suffix.js", array( 'jquery', 'hoverIntent', 'utils' ), false, 1 );
did_action( 'init' ) && $scripts->localize( $scripts->set_translations( 'common' );
'common',
'commonL10n',
array(
'warnDelete' => __( "You are about to permanently delete these items from your site.\nThis action cannot be undone.\n 'Cancel' to stop, 'OK' to delete." ),
'dismiss' => __( 'Dismiss this notice.' ),
'collapseMenu' => __( 'Collapse Main menu' ),
'expandMenu' => __( 'Expand Main menu' ),
)
);
$scripts->add( 'wp-sanitize', "/wp-includes/js/wp-sanitize$suffix.js", array(), false, 1 ); $scripts->add( 'wp-sanitize', "/wp-includes/js/wp-sanitize$suffix.js", array(), false, 1 );
@ -809,8 +800,8 @@ function wp_default_scripts( $scripts ) {
// Masonry v2 depended on jQuery. v3 does not. The older jquery-masonry handle is a shiv. // Masonry v2 depended on jQuery. v3 does not. The older jquery-masonry handle is a shiv.
// It sets jQuery as a dependency, as the theme may have been implicitly loading it this way. // It sets jQuery as a dependency, as the theme may have been implicitly loading it this way.
$scripts->add( 'imagesloaded', '/wp-includes/js/imagesloaded.min.js', array(), '4.1.4', 1 ); $scripts->add( 'imagesloaded', '/wp-includes/js/imagesloaded.min.js', array(), '3.2.0', 1 );
$scripts->add( 'masonry', '/wp-includes/js/masonry.min.js', array( 'imagesloaded' ), '4.2.2', 1 ); $scripts->add( 'masonry', '/wp-includes/js/masonry.min.js', array( 'imagesloaded' ), '3.3.2', 1 );
$scripts->add( 'jquery-masonry', "/wp-includes/js/jquery/jquery.masonry$dev_suffix.js", array( 'jquery', 'masonry' ), '3.1.2b', 1 ); $scripts->add( 'jquery-masonry', "/wp-includes/js/jquery/jquery.masonry$dev_suffix.js", array( 'jquery', 'masonry' ), '3.1.2b', 1 );
$scripts->add( 'thickbox', '/wp-includes/js/thickbox/thickbox.js', array( 'jquery' ), '3.1-20121105', 1 ); $scripts->add( 'thickbox', '/wp-includes/js/thickbox/thickbox.js', array( 'jquery' ), '3.1-20121105', 1 );

View File

@ -13,7 +13,7 @@
* *
* @global string $wp_version * @global string $wp_version
*/ */
$wp_version = '5.5-alpha-48266'; $wp_version = '5.5-alpha-48267';
/** /**
* Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema. * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.