[wp-trac] [WordPress Trac] #16750: Improve inline docs for l10n functions

WordPress Trac noreply at wordpress.org
Fri Sep 20 18:43:36 UTC 2013


#16750: Improve inline docs for l10n functions
------------------------------+-----------------------------
 Reporter:  CharlesClarkson   |       Owner:  SergeyBiryukov
     Type:  enhancement       |      Status:  closed
 Priority:  normal            |   Milestone:  3.7
Component:  Inline Docs       |     Version:  3.1
 Severity:  trivial           |  Resolution:  fixed
 Keywords:  has-patch commit  |
------------------------------+-----------------------------

Comment (by ericlewis):

 Replying to [comment:14 SergeyBiryukov]:
 > s/domain/text domain/ seems fine.
 >
 > For the longer string, I'd suggest adding "Text domain" in front of it,
 rather than replacing it. There should still be some short description,
 and the "unique identifier" serves that well.

 Here I thought giving the entire name for the unique identifier (Text
 domain) is more helpful instead of telling that is a unique identifier.
 There are a lot of unique identifiers that get tossed around WP core (Post
 ID, taxonomy name, etc.), and developers can be expected to look up the
 l18n API to understand what a text domain is, and that a text domain
 should be a unique identifier, which is why I left it out. It's a grey
 area between where we expect developers to seek more complete
 documentation and brevity.

 Please tell me if I'm getting too pedantic :)

--
Ticket URL: <http://core.trac.wordpress.org/ticket/16750#comment:15>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list