[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