[wp-trac] [WordPress Trac] #47695: i18n/multisite: Contextualize strings in theme update screen

WordPress Trac noreply at wordpress.org
Sat Jul 13 16:32:55 UTC 2019


#47695: i18n/multisite: Contextualize strings in theme update screen
-------------------------+-----------------------------
 Reporter:  audrasjb     |      Owner:  (none)
     Type:  enhancement  |     Status:  new
 Priority:  normal       |  Milestone:  Awaiting Review
Component:  I18N         |    Version:
 Severity:  normal       |   Keywords:  needs-patch
  Focuses:  multisite    |
-------------------------+-----------------------------
 In `wp-admin/includes/class-wp-ms-themes-list-table.php` file, under
 `get_views` function, only the first translatable screen is
 contextualized. Then, we have a gender issue in fr_FR with the other
 translatable strings. Thant would be nice to add a context to the other
 items as well.

 {{{#!php
 switch ( $type ) {
         case 'all':
                 $text = _nx( 'All <span class="count">(%s)</span>', 'All
 <span class="count">(%s)</span>', $count, 'themes' );
                 break;
         case 'enabled':
                 $text = _n( 'Enabled <span class="count">(%s)</span>',
 'Enabled <span class="count">(%s)</span>', $count );
                 break;
         case 'disabled':
                 $text = _n( 'Disabled <span class="count">(%s)</span>',
 'Disabled <span class="count">(%s)</span>', $count );
                 break;
         case 'upgrade':
                 $text = _n( 'Update Available <span
 class="count">(%s)</span>', 'Update Available <span
 class="count">(%s)</span>', $count );
                 break;
         case 'broken':
                 $text = _n( 'Broken <span class="count">(%s)</span>',
 'Broken <span class="count">(%s)</span>', $count );
                 break;
 }
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/47695>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list