[wp-trac] [WordPress Trac] #18887: Appearnce -> Header UI: "Display Text" Needs Distinct theme_mod

WordPress Trac wp-trac at lists.automattic.com
Sat Oct 8 19:45:46 UTC 2011


#18887: Appearnce -> Header UI: "Display Text" Needs Distinct theme_mod
--------------------------+------------------------------
 Reporter:  chipbennett   |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  UI            |     Version:  3.2
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by chipbennett):

 Replying to [comment:9 scribu]:
 > The function description is too verbose. I would replace it with just
 >
 > {{{
 > Conditional check to determine if the user has enabled the Display Text
 option.
 > }}}
 >
 > The function implementation is also too verbose. It should be a single
 line:
 >
 > {{{
 > return 'blank' != get_header_textcolor();
 > }}}

 Made less-verbose, as per your suggestion.

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


More information about the wp-trac mailing list