WordPress.org

Make WordPress Core

#24547 closed defect (bug) (fixed)

"Go to All %s" string not ready for i18n

Reported by: xibe Owned by: azaozz
Milestone: 3.6 Priority: high
Severity: major Version: 3.6
Component: I18N Keywords: 2nd-opinion
Focuses: Cc:

Description

The "Go to All %s" string (location is marked with %s being any post type: "Posts, Pages, etc.".

As it is, translators cannot do a direct translation of it: as with any "any post type"-related string, it should be known that depending on the language, some post types can have a "gender", and therefore cannot use the same sentence structure.

For instance, in French, Post is Article, which is masculine, when Page is Page, which is feminine. Translation for this string would then either be "Aller à tous les articles" or "Aller à toutes les pages".

Either the string is reworked, or translators should be advised to use a sentence structure that works for any content type.

Attachments (2)

locked.png (12.8 KB) - added by azaozz 10 months ago.
24547.patch (867 bytes) - added by azaozz 10 months ago.

Download all attachments as: .zip

Change History (7)

comment:1 nacin10 months ago

  • Milestone changed from Awaiting Review to 3.6
  • Priority changed from normal to high
  • Severity changed from normal to major

comment:2 azaozz10 months ago

One way to fix this would be to drop the "Go to" part and use get_post_type_object( $post->post_type )->labels->all_items;. The button will have exactly the same action and label as the menu.

azaozz10 months ago

comment:3 azaozz10 months ago

Alternatively can more the "Go to" part before the button where "Go" can be "Navigate" or "Move", etc.


In the plugin attached to #18515 the same button is called GET ME OUT :)

Other ideas/suggestions?

Last edited 10 months ago by azaozz (previous) (diff)

azaozz10 months ago

comment:4 SergeyBiryukov10 months ago

Introduced in [24408].

Yes, we should avoid post type and taxonomy names in generic strings, as noted in ticket:17609:3 and ticket:19099:1.

24547.patch looks good to me.

comment:5 azaozz10 months ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 24527:

Post locked dialog: remove the "Go to" part from the button as it's not translatable. Fixes #24547.

Note: See TracTickets for help on using tickets.