[wp-trac] [WordPress Trac] #14981: Provide a context for post statuses
WordPress Trac
noreply at wordpress.org
Thu Mar 3 22:35:16 UTC 2016
#14981: Provide a context for post statuses
-------------------------------------------------+-------------------------
Reporter: xibe | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future
Component: I18N | Release
Severity: normal | Version: 3.1
Keywords: good-first-bug has-patch dev- | Resolution:
feedback | Focuses:
-------------------------------------------------+-------------------------
Comment (by swissspidy):
Replying to [comment:36 ocean90]:
> [attachment:14981.3.patch]: I don't think we should split the string
`All <span class="count">(%s)</span>`, see
https://core.trac.wordpress.org/ticket/31859 and
https://core.trac.wordpress.org/ticket/31251#comment:17.
Agree, though it makes things a bit inconsistent. For example, the string
`Published` is needed both with the `<span>` and without it. What should
we do about that?
See: #31859, [ticket:31251#comment:17]
----
[attachment:14981.diff] is an updated patch without the splitted `All
<span class="count">(%s)</span>` string. I also removed the `auto-draft`
string as it's not needed (maps to `draft`) and fixed some wording
(`Sticky` is still `Sticky` in plural, not `Stickies`).
As for testing the changes, I ran `php tools/i18n/makepot.php wp-frontend
src test.pot` and the strings get correctly added to the POT file with the
different contexts.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/14981#comment:37>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list