Make WordPress Core

Opened 10 years ago

Closed 7 years ago

#38882 closed task (blessed) (duplicate)

Add translator comments to every string that contains a placeholder

Reported by: johnbillion Owned by: johnbillion
Priority: normal Milestone:
Component: I18N Version:
Severity: normal Keywords: needs-patch
Cc: Focuses:

Description

Strings that contain placeholders — such as %s, %d, and %1$s — should include accompanying translator comments. This provides context to translators and allows for more accurate translations.

The meaning of even seemingly simple strings containing placeholders may not be clear to translators who are not intimately familiar with WordPress, unless a translator comment is provided.

Change History (10)

#1 @johnbillion
10 years ago

  • Owner set to johnbillion
  • Status newassigned

#3 @johnbillion
10 years ago

In 39323:

I18n: Begin introducing translator comments for strings which include placeholders but no accompanying translator comment.

Adds context to one string used in two different contexts for the new user and new site signup email notification.

More to come.

See #38882

#4 @johnbillion
10 years ago

In 39324:

Remove an erroneous @TODO introduced in [39323].

See #38882

#5 @johnbillion
10 years ago

In 39326:

I18n: Introduce more translator comments for strings that contain placeholders but don't have an accompanying translator comment.

See #38882

#6 @ocean90
10 years ago

@johnbillion Comments which are for the placeholders %s or %d shouldn't use "1", it should be the name of the placeholder.

#7 @desrosj
7 years ago

  • Keywords needs-patch reporter-feedback added

@johnbillion Are you able to wrap this up or summarize what is left? Also, it appears that @ocean90's concern still needs to be addressed.

#8 @johnbillion
7 years ago

  • Keywords reporter-feedback removed
  • Milestone Future Release5.3

#10 @johnbillion
7 years ago

  • Milestone 5.3
  • Resolutionduplicate
  • Status assignedclosed

Superseded by #44360.

Note: See TracTickets for help on using tickets.