i18n: Add translators comment for placeholders used in "review queue" message

[felipeelia]

I've just noticed this while translating some strings of plugins directory. Before the _n in (plugin-directory/shortcodes/class-upload.php:70):

<?php
printf(
        _n(
                'Currently there are %1$s plugins in the review queue, %2$s of which is awaiting its initial review.',
                'Currently there are %1$s plugins in the review queue, %2$s of which are awaiting their initial review.',
                $plugins->new,
                'wporg-plugins'
        ),
        '<strong>' . ( $plugins->new + $plugins->pending ) . '</strong>',
        '<strong>' . $plugins->new . '</strong>'
);

we should have

/* translators: 1: new + pending plugins; 2: new plugins */

or something like that, shouldn't we?

[Otto42]

Comments are meant to explain things that are not clear. Is this unclear?

We don't add comments for every case of multiple strings, just the stranger cases.

[danieltj]

Replying to Otto42:

Comments are meant to explain things that are not clear. Is this unclear?

We don't add comments for every case of multiple strings, just the stranger cases.

I've seen less stranger cases with comments. Agreed that over explaining doesn't help, but in this case scenario, considering that there's more than one placeholder - it could only help.

[felipeelia]

Sorry @Otto42 and @danieltj,

Actually the _n call that is more misleading to me is this one, at line 91:

<?php
_n(
                'You have %1$s plugins in the review queue, %2$s is being actively reviewed. Please wait for them to be approved before submitting any more.',
                'You have %1$s plugins in the review queue, %2$s are being actively reviewed. Please wait for them to be approved before submitting any more.',
                $submitted_counts->pending,
                'wporg-plugins'
),
'<strong>' . $submitted_counts->total . '</strong>',
'<strong>' . $submitted_counts->pending . '</strong>'

English isn't my first language, but I think that a hint for what "actively reviewed" means in this case would help.

[Otto42]

I'm happy to add a comment, I just don't know what to add. What is the confusion, exactly?

[obenland]

In 6405:

Plugins: Add translator comments.

Provides more context for strings with placeholders.

Props felipeelia.
Fixes #3204.