[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:41:28 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 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();
}}}
--
Ticket URL: <http://core.trac.wordpress.org/ticket/18887#comment:9>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list