[wp-trac] [WordPress Trac] #44340: Use sprintf() to remove <strong> tags from translating strings

WordPress Trac noreply at wordpress.org
Tue Jun 7 21:27:14 UTC 2022

#44340: Use sprintf() to remove <strong> tags from translating strings
 Reporter:  saimonh      |       Owner:  (none)
     Type:  enhancement  |      Status:  closed
 Priority:  normal       |   Milestone:
Component:  I18N         |     Version:  4.9.6
 Severity:  normal       |  Resolution:  wontfix
 Keywords:               |     Focuses:  coding-standards
Changes (by SergeyBiryukov):

 * keywords:  close =>
 * status:  new => closed
 * resolution:   => wontfix
 * milestone:  Awaiting Review =>


 Hi there, thanks for the ticket!

 As noted in comment:2, it is normal for strings to have some basic HTML in
 them. Splitting strings into smaller chunks often leads to losing
 important context and making them less translatable without an obvious
 benefit. Complete sentences, on the other hand, are much easier to

 Looking at the patch, the current code:
         /* translators: %s: User name. */
         __( '<strong>ERROR</strong>: The password you entered for the
 username %s is incorrect.' ),
         '<strong>' . $username . '</strong>'
 is significantly more i18n-friendly than the proposed alternative:
         '<strong>%s</strong>: %s <strong>%s</strong> %s',
         __( 'ERROR' ),
         __( 'The password you entered for the username' ),
         __( 'is incorrect.' )

 Note that in the former example, the second `<strong>` tag is already
 moved out of the translatable string, as part of the previous efforts on
 removing unnecessary HTML from strings]. This is a balance between
 allowing some basic HTML to preserve complete sentences and moving any
 extra HTML out of translations.

Ticket URL: <https://core.trac.wordpress.org/ticket/44340#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform

More information about the wp-trac mailing list