WordPress.org

Make WordPress Core

Opened 20 months ago

Closed 19 months ago

Last modified 19 months ago

#35672 closed defect (bug) (fixed)

Avoid using HTML tags in translation strings (wp-admin/user-edit.php)

Reported by: ramiy Owned by: SergeyBiryukov
Milestone: 4.5 Priority: normal
Severity: normal Version:
Component: I18N Keywords: has-patch
Focuses: Cc:

Description

See the attached patch.

Attachments (2)

35672.patch (844 bytes) - added by ramiy 20 months ago.
35672.2.patch (863 bytes) - added by ramiy 20 months ago.

Download all attachments as: .zip

Change History (9)

@ramiy
20 months ago

#1 @ramiy
20 months ago

  • Keywords has-patch added

In this patch I've moved the <a href="%2$s">Cancel</a> part into a separate translation string. And also I've added a translator comment to the original string.

#2 @ramiy
20 months ago

Related: #35673

#3 @ramiy
20 months ago

Old string:

  • There is a pending change of your email to %1$s. <a href="%2$s">Cancel</a>

New strings:

  • There is a pending change of your email to %s.
  • Cancel

This ticket was mentioned in Slack in #core-i18n by ramiy. View the logs.


20 months ago

@ramiy
20 months ago

#5 @ramiy
20 months ago

The patch also adds translators comment.

#6 @SergeyBiryukov
19 months ago

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

In 36655:

I18N: Remove <a> tag from translatable string in wp-admin/user-edit.php.

Add translator comment.

Props ramiy.
Fixes #35672.

#7 @SergeyBiryukov
19 months ago

  • Milestone changed from Awaiting Review to 4.5
Note: See TracTickets for help on using tickets.