WordPress.org

Make WordPress Core

Changeset 48391


Ignore:
Timestamp:
07/07/2020 06:00:37 PM (10 months ago)
Author:
ocean90
Message:

I18N: Use wp.i18n for translatable strings in wp-admin/js/inline-edit-post.js and wp-admin/js/inline-edit-tax.js.

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

Props swissspidy, ocean90.
See #20491.
Fixes #50601.

Location:
trunk/src
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/js/_enqueues/admin/inline-edit-post.js

    r48318 r48391  
    66 */
    77
    8 /* global inlineEditL10n, ajaxurl, typenow, inlineEditPost */
     8/* global ajaxurl, typenow, inlineEditPost */
    99
    1010window.wp = window.wp || {};
     
    199199                c = false;
    200200                var id = $(this).val(), theTitle;
    201                 theTitle = $('#inline_'+id+' .post_title').html() || inlineEditL10n.notitle;
    202                 te += '<div id="ttle'+id+'"><a id="_'+id+'" class="ntdelbutton" title="'+inlineEditL10n.ntdeltitle+'">X</a>'+theTitle+'</div>';
     201                theTitle = $('#inline_'+id+' .post_title').html() || wp.i18n.__( '(no title)' );
     202                te += '<div id="ttle'+id+'"><a id="_'+id+'" class="ntdelbutton" title="'+ wp.i18n.__( 'Remove From Bulk Edit' ) +'">X</a>'+theTitle+'</div>';
    203203            }
    204204        });
     
    328328                taxname = $(this).attr('id').replace('_' + id, ''),
    329329                textarea = $('textarea.tax_input_' + taxname, editRow),
    330                 comma = inlineEditL10n.comma;
     330                comma = wp.i18n._x( ',', 'tag delimiter' ).trim();
    331331
    332332            terms.find( 'img' ).replaceWith( function() { return this.alt; } );
     
    432432                                .attr( 'aria-expanded', 'false' )
    433433                                .focus();
    434                             wp.a11y.speak( inlineEditL10n.saved );
     434                            wp.a11y.speak( wp.i18n.__( 'Changes saved.' ) );
    435435                        });
    436436                    } else {
     
    442442                } else {
    443443                    $errorNotice.removeClass( 'hidden' );
    444                     $error.html( inlineEditL10n.error );
    445                     wp.a11y.speak( inlineEditL10n.error );
     444                    $error.text( wp.i18n.__( 'Error while saving the changes.' ) );
     445                    wp.a11y.speak( wp.i18n.__( 'Error while saving the changes.' ) );
    446446                }
    447447            },
  • trunk/src/js/_enqueues/admin/inline-edit-tax.js

    r47122 r48391  
    55 */
    66
    7 /* global inlineEditL10n, ajaxurl, inlineEditTax */
     7/* global ajaxurl, inlineEditTax */
    88
    99window.wp = window.wp || {};
     
    225225                                .attr( 'aria-expanded', 'false' )
    226226                                .focus();
    227                             wp.a11y.speak( inlineEditL10n.saved );
     227                            wp.a11y.speak( wp.i18n.__( 'Changes saved.' ) );
    228228                        });
    229229
     
    239239                } else {
    240240                    $errorNotice.removeClass( 'hidden' );
    241                     $error.html( inlineEditL10n.error );
    242                     wp.a11y.speak( inlineEditL10n.error );
     241                    $error.text( wp.i18n.__( 'Error while saving the changes.' ) );
     242                    wp.a11y.speak( wp.i18n.__( 'Error while saving the changes.' ) );
    243243                }
    244244            }
  • trunk/src/wp-includes/script-loader.php

    r48387 r48391  
    12551255
    12561256        $scripts->add( 'inline-edit-post', "/wp-admin/js/inline-edit-post$suffix.js", array( 'jquery', 'tags-suggest', 'wp-a11y' ), false, 1 );
    1257         did_action( 'init' ) && $scripts->localize(
    1258             'inline-edit-post',
    1259             'inlineEditL10n',
    1260             array(
    1261                 'error'      => __( 'Error while saving the changes.' ),
    1262                 'ntdeltitle' => __( 'Remove From Bulk Edit' ),
    1263                 'notitle'    => __( '(no title)' ),
    1264                 'comma'      => trim( _x( ',', 'tag delimiter' ) ),
    1265                 'saved'      => __( 'Changes saved.' ),
    1266             )
    1267         );
     1257        $scripts->set_translations( 'inline-edit-post' );
    12681258
    12691259        $scripts->add( 'inline-edit-tax', "/wp-admin/js/inline-edit-tax$suffix.js", array( 'jquery', 'wp-a11y' ), false, 1 );
    1270         did_action( 'init' ) && $scripts->localize(
    1271             'inline-edit-tax',
    1272             'inlineEditL10n',
    1273             array(
    1274                 'error' => __( 'Error while saving the changes.' ),
    1275                 'saved' => __( 'Changes saved.' ),
    1276             )
    1277         );
     1260        $scripts->set_translations( 'inline-edit-tax' );
    12781261
    12791262        $scripts->add( 'plugin-install', "/wp-admin/js/plugin-install$suffix.js", array( 'jquery', 'jquery-ui-core', 'thickbox' ), false, 1 );
Note: See TracChangeset for help on using the changeset viewer.