Opened 9 years ago
Closed 8 years ago
#36336 closed defect (bug) (fixed)
Update docblock for get_header_textcolor() and header_textcolor()
Reported by: | theMikeD | Owned by: | DrewAPicture |
---|---|---|---|
Milestone: | 4.8 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Themes | Keywords: | has-patch |
Focuses: | docs | Cc: |
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
@
9 years ago
- Component changed from General to Themes
- Keywords has-patch added
- Milestone changed from Awaiting Review to Future Release
- Owner set to DrewAPicture
- Status changed from new to reviewing
#2
@
9 years ago
- Keywords commit added
- Milestone changed from Future Release to 4.5
- Type changed from enhancement to defect (bug)
#5
follow-up:
↓ 6
@
9 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
@
9 years ago
- Keywords commit removed
- Milestone changed from 4.5 to Future Release
- Resolution fixed deleted
- Status changed from closed to reopened
Replying to swissspidy:
I don't really like
HEX format
(why uppercase anyway?).6-digit hexadecimal form
sounds 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.
Note: See
TracTickets for help on using
tickets.
@azaozz Can you please review the changes in 36336.2.diff?