WordPress.org

Make WordPress Core

Opened 3 months ago

Closed 3 months ago

Last modified 7 weeks ago

#43523 closed task (blessed) (fixed)

Translator comment improvements for 5.0

Reported by: SergeyBiryukov Owned by: SergeyBiryukov
Milestone: 5.0 Priority: normal
Severity: normal Version:
Component: I18N Keywords:
Focuses: Cc:

Description

Background: #41974

Some translator comments have a number, but the corresponding string does not use numbered placeholders:

/* translators: 1: Site name */
printf( __( 'Links for %s' ), esc_attr( get_bloginfo( 'name', 'display' ) ) );

The comment should reference the actual placeholder instead:

/* translators: %s: site name */

Change History (4)

#1 @SergeyBiryukov
3 months ago

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

In 42827:

I18N: Use the actual placeholder instead of a number in translator comments if the corresponding string does not use numbered placeholders.

Add missing translator comments in WP_Theme_Install_List_Table and wp_notify_postauthor().
Add missing commas in some translator comments.

Fixes #43523.

#2 @SergeyBiryukov
2 months ago

In 42983:

I18N: Correct placeholder references in a translator comment in wp_print_file_editor_templates().

Props afercia.
See #43523.

#3 @SergeyBiryukov
2 months ago

In 42984:

I18N: Use consistent pattern for placeholder references in a translator comment in media_upload_max_image_resize().

See #43523.

#4 @SergeyBiryukov
7 weeks ago

In 43090:

I18N: Use consistent pattern for placeholder references in a translator comment in wp-admin/update-core.php.

See #43523.

Note: See TracTickets for help on using tickets.