Opened 9 years ago
Closed 9 years ago
#2881 closed defect (bug) (fixed)
Avoid using HTML tags in translation strings
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Locale - i18n/l10n | Keywords: | has-patch commit has-screenshots |
Cc: |
Description
In the next version of WordPress (4.4), we fixed many translation strings that used HTML tags, removing the HTML tags out from the strings.
The same thing done in BuddyPress 2.4.
In this ticket I will try to do the same for bbPress. See the attached patches.
Attachments (6)
Change History (13)
#1
follow-up:
↓ 2
@
9 years ago
- Keywords commit has-screenshots added
- Milestone changed from Awaiting Review to 2.6
Thanks for these as well! Added 2881.03.patch to condense patches into one. Fixes issues with 2881-strong.patch where the <strong>
tags were added to the wrong string, and some code formatting issues.
#2
in reply to:
↑ 1
@
9 years ago
Replying to thebrandonallen:
Thanks for these as well! Added 2881.03.patch to condense patches into one. Fixes issues with 2881-strong.patch where the
<strong>
tags were added to the wrong string, and some code formatting issues.
Thanks for fixing the patch.
I have just one request, it seems like the sprintf() function has 3 parameters (is has only two).
We need to set the date_i18n() function with it's parameters in one line. This way, the code wont be confusing.
move <em> tags out of the string