[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