WordPress.org

Make WordPress Core


Ignore:
Timestamp:
09/01/2019 05:12:43 PM (7 weeks ago)
Author:
SergeyBiryukov
Message:

I18N: Improve translator comments.

  • Add missing translator comments.
  • Fix placement of some translator comments. Translator comments should be on the line directly above the line containing the translation function call for optimal compatibility with various .pot file generation tools. The CS auto-fixing, which changed some inconsistent function calls to multi-line function calls, is part of the reason why this was no longer the case for a select group of translator comments.

Includes minor code layout fixes.

Polyglots, rejoice! All WordPress core files now have translator comments for all strings with placeholders!

Props jrf, subrataemfluence, GaryJ, webdados, Dency, swissspidy, alvarogois, marcomartins, mihaiiceyro, vladwtz, niq1982, flipkeijzer, michielatyoast, chandrapatel, thrijith, joshuanoyce, FesoVik, tessak22, bhaktirajdev, cleancoded, dhavalkasvala, garrett-eclipse, bibliofille, socalchristina, priyankkpatel, 5hel2l2y, adamsilverstein, JeffPaul, pierlo, SergeyBiryukov.
Fixes #44360.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/wp-admin/includes/class-wp-plugins-list-table.php

    r45905 r45926  
    430430            switch ( $type ) {
    431431                case 'all':
    432                     /* translators: %s: plugin count */
    433                     $text = _nx( 'All <span class="count">(%s)</span>', 'All <span class="count">(%s)</span>', $count, 'plugins' );
     432                    /* translators: %s: number of plugins */
     433                    $text = _nx(
     434                        'All <span class="count">(%s)</span>',
     435                        'All <span class="count">(%s)</span>',
     436                        $count,
     437                        'plugins'
     438                    );
    434439                    break;
    435440                case 'active':
    436                     /* translators: %s: plugin count */
    437                     $text = _n( 'Active <span class="count">(%s)</span>', 'Active <span class="count">(%s)</span>', $count );
     441                    /* translators: %s: number of plugins */
     442                    $text = _n(
     443                        'Active <span class="count">(%s)</span>',
     444                        'Active <span class="count">(%s)</span>',
     445                        $count
     446                    );
    438447                    break;
    439448                case 'recently_activated':
    440                     /* translators: %s: plugin count */
    441                     $text = _n( 'Recently Active <span class="count">(%s)</span>', 'Recently Active <span class="count">(%s)</span>', $count );
     449                    /* translators: %s: number of plugins */
     450                    $text = _n(
     451                        'Recently Active <span class="count">(%s)</span>',
     452                        'Recently Active <span class="count">(%s)</span>',
     453                        $count
     454                    );
    442455                    break;
    443456                case 'inactive':
    444                     /* translators: %s: plugin count */
    445                     $text = _n( 'Inactive <span class="count">(%s)</span>', 'Inactive <span class="count">(%s)</span>', $count );
     457                    /* translators: %s: number of plugins */
     458                    $text = _n(
     459                        'Inactive <span class="count">(%s)</span>',
     460                        'Inactive <span class="count">(%s)</span>',
     461                        $count
     462                    );
    446463                    break;
    447464                case 'mustuse':
    448                     /* translators: %s: plugin count */
    449                     $text = _n( 'Must-Use <span class="count">(%s)</span>', 'Must-Use <span class="count">(%s)</span>', $count );
     465                    /* translators: %s: number of plugins */
     466                    $text = _n(
     467                        'Must-Use <span class="count">(%s)</span>',
     468                        'Must-Use <span class="count">(%s)</span>',
     469                        $count
     470                    );
    450471                    break;
    451472                case 'dropins':
    452                     /* translators: %s: plugin count */
    453                     $text = _n( 'Drop-in <span class="count">(%s)</span>', 'Drop-ins <span class="count">(%s)</span>', $count );
     473                    /* translators: %s: number of plugins */
     474                    $text = _n(
     475                        'Drop-in <span class="count">(%s)</span>',
     476                        'Drop-ins <span class="count">(%s)</span>',
     477                        $count
     478                    );
    454479                    break;
    455480                case 'paused':
    456                     /* translators: %s: plugin count */
    457                     $text = _n( 'Paused <span class="count">(%s)</span>', 'Paused <span class="count">(%s)</span>', $count );
     481                    /* translators: %s: number of plugins */
     482                    $text = _n(
     483                        'Paused <span class="count">(%s)</span>',
     484                        'Paused <span class="count">(%s)</span>',
     485                        $count
     486                    );
    458487                    break;
    459488                case 'upgrade':
    460                     /* translators: %s: plugin count */
    461                     $text = _n( 'Update Available <span class="count">(%s)</span>', 'Update Available <span class="count">(%s)</span>', $count );
     489                    /* translators: %s: number of plugins */
     490                    $text = _n(
     491                        'Update Available <span class="count">(%s)</span>',
     492                        'Update Available <span class="count">(%s)</span>',
     493                        $count
     494                    );
    462495                    break;
    463496            }
     
    769802            $checkbox = '';
    770803        } else {
    771             /* translators: %s: plugin name */
    772             $checkbox = "<label class='screen-reader-text' for='" . $checkbox_id . "' >" . sprintf( __( 'Select %s' ), $plugin_data['Name'] ) . '</label>'
    773                 . "<input type='checkbox' name='checked[]' value='" . esc_attr( $plugin_file ) . "' id='" . $checkbox_id . "' />";
     804            $checkbox = sprintf(
     805                '<label class="screen-reader-text" for="%1$s">%2$s</label>' .
     806                '<input type="checkbox" name="checked[]" value="%3$s" id="%1$s" />',
     807                $checkbox_id,
     808                /* translators: %s: plugin name */
     809                sprintf( __( 'Select %s' ), $plugin_data['Name'] ),
     810                esc_attr( $plugin_file )
     811            );
    774812        }
    775813        if ( 'dropins' != $context ) {
Note: See TracChangeset for help on using the changeset viewer.