Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#49306 closed enhancement (fixed)

Translation of the term post in the export tool

Reported by: jdy68's profile jdy68 Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.4 Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch has-screenshots
Focuses: administration Cc:

Description

We have identified an issue in the interface of the WordPress Export tool.
This issue is related to the translation of the term "Post" in our language (fr_FR).

The fr_FR locale team translates the term "post" differently depending on the context, as other locales.
When it refers to the "post" type item, we translate "post" by "post" (article in french) and when the term "post" refers to a more generic item that encompasses both the "post", "page" and "custom post types" items, we use another term ("Publication" in french) to highlight the difference between the "post" item and the generic items.

In the export tool "Posts" (see line 183 in wp-admin/export.php, below) has been mistakenly translated as "Publications" because a description of the context is missing.

<p><label><input type="radio" name="content" value="posts" /> <?php _e( 'Posts' ); ?></label></p>

Please add a context to this string.
Thanks.

Attachments (2)

49306.diff (929 bytes) - added by audrasjb 5 years ago.
i18n: add context to Posts string located in Export Tool screen.
outils-exporter.png (107.0 KB) - added by audrasjb 5 years ago.

Download all attachments as: .zip

Change History (8)

@audrasjb
5 years ago

i18n: add context to Posts string located in Export Tool screen.

#1 @audrasjb
5 years ago

  • Keywords has-patch added; needs-patch removed

Hi @jdy68 from the French translation team 🙂 …and welcome to WordPress Core Trac!

Indeed, this is an annoying issue for Locales that are disambiguating "Posts" as a specific Post type (News/Blog Articles) and "Posts" as a generic term to identify a publication on WordPress.

It looks pretty easy to fix, as we just need to use the same translation string used for the general name of the "Posts" post type declaration.

https://build.trac.wordpress.org/browser/trunk/wp-includes/post.php?marks=1683#L1683

_x( 'Posts', 'post type general name' )

In 49306.diff:

  • i18n: add context to Posts string located in Export Tool screen.

I believe we don't need to use contextualization for the other post types located in this screen as the issue only occurs with the English ambiguity of Posts/Posts.

#2 @audrasjb
5 years ago

  • Keywords has-screenshots added

#3 @johnbillion
5 years ago

  • Milestone changed from Awaiting Review to 5.4

#4 @audrasjb
5 years ago

@SergeyBiryukov I now you're quite busy, but pinging you are you are one of this component maintainers: any chance to get this one reviewed for 5.4?

It's a very small patch but it would provide a very nice enhancement for localized versions of WP. Thanks!

#5 @SergeyBiryukov
5 years ago

  • Owner set to SergeyBiryukov
  • Status changed from new to reviewing

#6 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from reviewing to closed

In 47272:

Export: Use correct context for the Posts label on the Export screen.

The string refers to the post type name specifically, not to a generic term to identify a publication on WordPress.

Props audrasjb, jdy68.
Fixes #49306.

Note: See TracTickets for help on using tickets.