Opened 11 years ago
Closed 10 years ago
#36336 closed defect (bug) (fixed)
Update docblock for get_header_textcolor() and header_textcolor()
| Reported by: | theMikeD | Owned by: | DrewAPicture |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.8 |
| Component: | Themes | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | docs |
Description
These two functions return the colour in HEX but that is not indicated in the docblock. Since we are pulling the docs from code, this should be added.
Attachments (3)
Change History (13)
#1
@
11 years ago
- Component General → Themes
- Keywords has-patch added
- Milestone Awaiting Review → Future Release
- Owner set to
- Status new → reviewing
#2
@
11 years ago
- Keywords commit added
- Milestone Future Release → 4.5
- Type enhancement → defect (bug)
#5
follow-up:
↓ 6
@
11 years ago
I don't really like HEX format (why uppercase anyway?). 6-digit hexadecimal form sounds much better. See twentyfifteen_hex2rgb() for an example.
#6
in reply to: ↑ 5
@
11 years ago
- Keywords commit removed
- Milestone 4.5 → Future Release
- Resolution fixed
- Status closed → reopened
Replying to swissspidy:
I don't really like
HEX format(why uppercase anyway?).6-digit hexadecimal formsounds much better. Seetwentyfifteen_hex2rgb()for an example.
I can get on board with "hexadecimal form" or similar. Do hex colors always save as 6 digits? I seem to recall a case where a default could be supplied as 3-digits and return as such, might've been fixed.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
@azaozz Can you please review the changes in 36336.2.diff?