[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