Opened 12 years ago
Closed 12 years ago
#22306 closed defect (bug) (fixed)
Some other string tweaks
Reported by: | pavelevap | Owned by: | nacin |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 3.4 |
Component: | I18N | Keywords: | has-patch 3.6-early commit |
Focuses: | Cc: |
Description
There are some other string tweaks: removing duplicate strings, removing HTML from strings, adding
"WordPress ›" to title as usual, using the same string where possible, etc. As a result, there are less strings to translate and several shorter strings to translate. Please see attached patch for detailed ideas.
Attachments (5)
Change History (21)
#4
@
12 years ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In 22430:
#5
@
12 years ago
I left out a few changes:
- Did not see the need to add "a" to the ubiquitous "error establishing database connection" message.
- I didn't want to break a bunch of long and long-standing strings a month left in the cycle just to remove some HTML from them.
#6
@
12 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
- Did not see the need to add "a" to the ubiquitous "error establishing database connection" message.
Because we have the same string with "a" and without "a". I removed "a" in other places in updated patch.
- I didn't want to break a bunch of long and long-standing strings a month left in the cycle just to remove some HTML from them.
I do not think that one month is a big problem for translators. These strings contain the same sentences which can be then removed from localization files and there will be less work for translators in the end (only some of them were changed by removing HTML). I understand your reasons, but in this case we should not close this ticket?
I added some other little tweak in updated patch (removing double space).
#10
@
12 years ago
- Cc xoodrew@… added
Looks like I'm late to the party, but I'd argue that use of <strong>
or <em>
tags inside a translatable string provides valid context for what requires emphasis.
I believe @otto42 may have touched on this in a recent blog post though this instance may not fall into what he'd consider a "rare" case where emphasis is warranted.
#11
follow-up:
↓ 14
@
12 years ago
- Keywords 3.6-early added
Per the IRC chat, 22306.4.patch can be done early next cycle.
#14
in reply to:
↑ 11
@
12 years ago
Replying to SergeyBiryukov:
Did you still want to try to get this in for 3.6 or again wait for 3.7 early?
Patch is simple enough +1 for commit.