Opened 11 years ago
Closed 11 years ago
#23585 closed enhancement (fixed)
Twenty Thirteen: Theme customizer: transport => postMessage for header_textcolor
Reported by: | SriniG | Owned by: | lancewillett |
---|---|---|---|
Milestone: | 3.6 | Priority: | normal |
Severity: | normal | Version: | 3.6 |
Component: | Bundled Theme | Keywords: | has-patch |
Focuses: | Cc: |
Description
Why refresh header_textcolor when we can have live js preview?
Attachments (3)
Change History (9)
#3
@
11 years ago
- Keywords needs-testing added
The first patch has a little problem. It works beautifully the first time, but after you set a color, save the changes and try it again, it doesn't work. Seems this is due to the !important
rule in custom-header. So removed the rule, and attachment:23585.1.diff is the consolidated patch. Not sure if removing the !important
would cause other problems.
#4
in reply to:
↑ 2
@
11 years ago
Replying to lancewillett:
Good idea, want to also look at adding this to Twenty Twelve?
Sure, I'm trying, but looks like it's a little complicated with Twenty Twelve.
@
11 years ago
Updating the 'version' parameter for wp_enqueue_script( 'twentythirteen-customizer',...)
Note: See
TracTickets for help on using
tickets.
Good idea, want to also look at adding this to Twenty Twelve?