Translation

[SGr33n]

Hi!

I'm translating in italian and found two spaces in this:
"Activation email resent! Please check your inbox or spam folder."

Then, I cannot understand this:

"<strong>Link</strong> - Activity generated by posts and comments, forum topics and replies, and some plugins, uses the link field for a permalink back to the content item. Some types of activity may not use this field, even if it has been set."

Then in cases like this (singular)
"Viewing post %1$s to %2$s (%3$s post)"

Could be better say:
"Viewing 1 post" ?

[SGr33n]

Also:
"You have %d new private messages" doesn't have a singular form.

[DJPaul]

Then in cases like this (singular)
"Viewing post %1$s to %2$s (%3$s post)"
Could be better say:
"Viewing 1 post" ?

I agree. If you can give us a list of all of these since I guess you can see them pretty quickly in GlotPress, we're happy to change them.

[djpaul]

In 8654:

i18n improvements

Simplified a sentence, removed a double space.

See #5767, props SGr33n

[djpaul]

In 8655:

i18n improvements in Message notifications

Add singular form of “you have x new private messages”, and use
bp_core_number_format to format the number correctly for the locale.

See #5767, props SGr33n

[SGr33n]

Replying to DJPaul:

I agree. If you can give us a list of all of these since I guess you can see them pretty quickly in GlotPress, we're happy to change them.

Sure! Here they are:

Viewing requests %1$s to %2$s (of %3$s request) bp-groups/bp-groups-template.php:3236
Viewing invitation %1$s to %2$s (of %3$s invitation) bp-groups/bp-groups-template.php:3531
Viewing member %1$s to %2$s (of %3$s member) bp-groups/bp-groups-template.php:2363 bp-members/bp-members-template.php:353
Viewing message %1$s to %2$s (of %3$s message) bp-messages/bp-messages-template.php:456
Viewing %1$s to %2$s (of %3$s notification) bp-notifications/bp-notifications-template.php:895
Viewing group %1$s to %2$s (of %3$s group) bp-groups/bp-groups-template.php:976
Viewing member %1$s to %2$s (of %3$s active member) bp-members/bp-members-template.php:347
Viewing member %1$s to %2$s (of %3$s member with friends) bp-members/bp-members-template.php:349
Viewing member %1$s to %2$s (of %3$s member online) bp-members/bp-members-template.php:351
Viewing topic %s to %s (of %d topic%s) bp-forums/bp-forums-template.php:1527
Viewing post %1$s to %2$s (%3$s post) bp-forums/bp-forums-template.php:2166
Viewing item %1$s to %2$s (of %3$s item) bp-activity/bp-activity-template.php:737
Viewing site %1$s to %2$s (of %3$s site) bp-blogs/bp-blogs-template.php:439
Viewing %1$s - %2$s of %3$s bp-groups/bp-groups-admin.php:942 - doesn't have a singular form

About plurals, what do you think about semplificate in this way "%1$s - %2$s of %3$s Messages"?

[DJPaul]

Thanks. I think I like the proposed simplification, too.

[SGr33n]

Well! :)

[DJPaul]

danbp - thanks for the patch, but in this case, we need to use the _nx function if you think adding a context to these strings is what needs to happen. I'm pretty sure you know of the function, but just in case you don't or someone else is following along with this ticket who doesn't, see: ​https://codex.wordpress.org/Function_Reference/_nx

[DJPaul]

I've just added 5767.02.patch which changes the singular and plural pagination strings as suggested by SGr33n. It doesn't add any contexts.

For the plural pagination strings, I have left the word "Viewing" in. For example, "Viewing 1 - 10 of 20 groups". Another one of SGr33n's suggestions was that we remove the word "Viewing" here, but keep "Viewing" in for the singular string. From my English POV, I think we could probably remove "Viewing" here, but I would like more opinions, and also whether "Viewing" should be kept in if it would help the string be translated (or appear better in the translated language).

[danbp]

Yes Paul i know about _nx function, sorry for this error.

IMO viewving is a secondary information near a counter. Not all sites/projects need this. Not every template has the place for that...
So adding context lets each translator add what he want, under the condition to use the variables order (%1$s - %2$s to %3$s).
If extended phrasing is needed above this simple output, it's under the responsability of the translator. (Hey folks, discover group %1$s to %2$s. And it's just the begin: we have a total of %3$s awaiting you !) Very long extreme usage. :-) But who knows ?

Why is the singular form ( viewing 1 item) more important as a plural (21 to 41 of 68) ?
If there is only one page, or member, or post, i already see it ! This doesn't even need a count.

Maybe showing the singular form as 1 post or 1 member or 1 something ( 1 + context) is more clear.
But if i go indepth with that logic, i would also remove any filter, because there is nothing to filter when the result is one.

I would use the most simple and minimalist option in the source (%1$s - %2$s of %3$s) and let each translator add his own wording accordingly to his project.

It's just an opinion.

[DJPaul]

If there is only one page, or member, or post, i already see it ! This doesn't even need a count.

It does because there's a difference in expecting to view one item (say, a single blog post) and expecting to view > 1 item (but only showing 1 because that's just how many things there are at the moment).

let each translator add his own wording accordingly to his project.

I agree with the gist of what you're saying but I think you're mixing up sites that use translations to customise the site, vs sites that use a translation just to get a translation. :)

[djpaul]

In 8780:

Groups: fix singular string translation in wp-admin members list pagination

See #5767

[djpaul]

In 8781:

Improve pagination label strings for all components

“Viewing 1 site” reads better than “Viewing site 1 to 1 (of 1 site)”,
regardless of the language you’re using.

Fixes #5767

[djpaul]

In 8782:

Legacy forums: correctly use an integer for _n calls when building the pagination labels

Previously, the number used was the (string) return value from
bp_core_number_format. That function uses number_format_i8n
internally, which can return values such as “1,000” — which would cause
_n to pick the wrong singular/plural string at runtime.

See #5767

[SGr33n]

Hi djpaul.

Maybe there is an error on bp-groups-template.php, ine 1545

                return apply_filters( 'bp_get_groups_pagination_count', sprintf( _n( 'Viewing 1', 'Viewing %1$s - %2$s of %3$s groups', $total, 'buddypress' ), $from_num, $to_num, $total ), $from_num, $to_num, $total );

Should be

                return apply_filters( 'bp_get_groups_pagination_count', sprintf( _n( 'Viewing 1 group', 'Viewing %1$s - %2$s of %3$s groups', $total, 'buddypress' ), $from_num, $to_num, $total ), $from_num, $to_num, $total );

[boonebgorges]

Yes, it looks like that was a slip of the finger in r8781.

[boonebgorges]

In 9036:

Pagination text for a groups loop with a single group should read "Viewing 1 group"

Fixes typo in r8781

Fixes #5767

Props SGr33n