[wp-trac] Re: [WordPress Trac] #5131: wp_tag_cloud(): title='1 topics', should be 1 topic

WordPress Trac wp-trac at lists.automattic.com
Tue Oct 2 17:28:04 GMT 2007


#5131: wp_tag_cloud(): title='1 topics', should be 1 topic
-------------------------+--------------------------------------------------
 Reporter:  atppp        |        Owner:  anonymous
     Type:  enhancement  |       Status:  new      
 Priority:  normal       |    Milestone:  2.5      
Component:  General      |      Version:  2.3      
 Severity:  normal       |   Resolution:           
 Keywords:               |  
-------------------------+--------------------------------------------------
Comment (by webrocker):

 Replying to [comment:3 Nazgul]:
 > This patch kills right-to-left language support.
 Ooops. Is this because of the way how the digit is 'handwired' in front of
 the 'topics' label (like '1 topic'), whereas a right-to-left needs
 something like 'topic 1'? In the original it is {{{sprintf( __('%d
 topics'), $count )}}}, so the digit is in front of the label anyway?
 Sorry, I don't get the difference between a hardcoded 'topics' after the
 %d and the variable used in my patch {{{sprintf( __('%d '.$topictxt),
 $count )}}}.

 Replying to [comment:4 foolswisdom]:
 > Please don't set the milestone, leave that for a core contributor.
 Ok, sorry for that.

 Replying to [comment:5 westi]:
 > Indeed, patch should be reworked to use
 [http://trac.wordpress.org/browser/trunk/wp-includes/l10n.php#L50
 __ngettext] rather than !__

 well the original used !__ , so I didn't messed with that.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/5131#comment:6>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list