Changeset 48383 for trunk/src/wp-includes/script-loader.php

Timestamp:

07/07/2020 04:07:25 PM (4 years ago)

Author:

ocean90

Message:

I18N: Use wp.i18n for translatable strings in wp-admin/js/color-picker.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 #50596.

File:

• trunk/src/wp-includes/script-loader.php (modified) (1 diff)

trunk/src/wp-includes/script-loader.php

@@ -1350 +1350 @@
         $scripts->add( 'iris', '/wp-admin/js/iris.min.js', array( 'jquery-ui-draggable', 'jquery-ui-slider', 'jquery-touch-punch' ), '1.0.7', 1 );
         $scripts->add( 'wp-color-picker', "/wp-admin/js/color-picker$suffix.js", array( 'iris' ), false, 1 );
-        did_action( 'init' ) && $scripts->localize(
-            'wp-color-picker',
-            'wpColorPickerL10n',
-            array(
-                'clear'            => __( 'Clear' ),
-                'clearAriaLabel'   => __( 'Clear color' ),
-                'defaultString'    => __( 'Default' ),
-                'defaultAriaLabel' => __( 'Select default color' ),
-                'pick'             => __( 'Select Color' ),
-                'defaultLabel'     => __( 'Color value' ),
-            )
-        );
+        $scripts->set_translations( 'wp-color-picker' );
 
         $scripts->add( 'dashboard', "/wp-admin/js/dashboard$suffix.js", array( 'jquery', 'admin-comments', 'postbox', 'wp-util', 'wp-a11y' ), false, 1 );