WordPress.org
Get WordPress

Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 5 years ago

#43523 closed task (blessed) (fixed)

Translator comment improvements for 5.1

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

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 (6)

#1 @SergeyBiryukov
9 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → 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
8 years ago

In 42983:

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

Props afercia.
See #43523.

#3 @SergeyBiryukov
8 years ago

In 42984:

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

See #43523.

#4 @SergeyBiryukov
8 years ago

In 43090:

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

See #43523.

#5 @johnbillion
8 years ago

  • Milestone 5.0 → 5.1

#6 @SergeyBiryukov
5 years ago

  • Summary Translator comment improvements for 5.0 → Translator comment improvements for 5.1
Note: See TracTickets for help on using tickets.