Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#47271 closed defect (bug) (fixed)

i18n: Avoid using HTML tags in translation strings (wp-includes/customize/class-wp-customize-theme-control.php)

Reported by: ramiy's profile ramiy Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 5.3 Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-screenshots has-patch
Focuses: administration Cc:

Description

See the attached patch.

Attachments (2)

47271.png (7.1 KB) - added by ramiy 5 years ago.
47271.patch (902 bytes) - added by ramiy 5 years ago.

Download all attachments as: .zip

Change History (5)

@ramiy
5 years ago

@ramiy
5 years ago

#1 @ramiy
5 years ago

  • Keywords has-screenshots has-patch added

Simpler string to translate in RTL languages where LTR code is making it hard to translate RTL letters.

The is an unwritten convention implemented all over the code. It should be implemented in this string too.

Also, in this case (<span>Previewing:</span> %s), translators can't really translate the HTML and the placeholder.

#2 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 5.3

#3 @SergeyBiryukov
5 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 45435:

I18N: Remove unnecessary HTML from "Previewing:" string in WP_Customize_Theme_Control.

Props ramiy.
Fixes #47271.

Note: See TracTickets for help on using tickets.