WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#23600 closed enhancement (fixed)

Twenty Twelve: Theme customizer: transport => postMessage for header_textcolor

Reported by: srinig's profile SriniG Owned by: lancewillett's profile lancewillett
Milestone: 3.6 Priority: normal
Severity: normal Version: 3.6
Component: Bundled Theme Keywords: needs-patch
Focuses: Cc:

Description

Live js preview for the header_textcolor option in the theme customizer.

Attachments (5)

23600.diff​ (2.4 KB) - added by SriniG 12 years ago.
23600.2.patch​ (2.1 KB) - added by lancewillett 12 years ago.
A few minor fixes, also align assignments
23600.2.diff​ (752 bytes) - added by SergeyBiryukov 12 years ago.
23600.3.diff​ (1.3 KB) - added by SergeyBiryukov 12 years ago.
23600.4.diff​ (1.0 KB) - added by obenland 12 years ago.

Download all attachments as: .zip

Change History (11)

@SriniG
12 years ago

#1 @SriniG
12 years ago

  • Component changed from General to Bundled Theme

#2 @lancewillett
12 years ago

  • Milestone changed from Awaiting Review to 3.6

@lancewillett
12 years ago

A few minor fixes, also align assignments

#3 @lancewillett
12 years ago

  • Owner set to lancewillett
  • Resolution set to fixed
  • Status changed from new to closed

In 23572:

Twenty Twelve: enable live preview for Header Text Color in Theme Customizer. Change also removes an !important declaration (yay) in Custom Header CSS rules.

Props SriniG, fixes #23600.

@SergeyBiryukov
12 years ago

#4 @SergeyBiryukov
12 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This introduced a bug with "Display Header Text" checkbox, see #23722.

@SergeyBiryukov
12 years ago

#5 @lancewillett
12 years ago

  • Keywords needs-patch added; has-patch removed
  • Severity changed from minor to normal

@obenland
12 years ago

#6 @lancewillett
12 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 23711:

Twenty Twelve: fix regression introduced in r23572, allow hide/show of Header Text in Customizer. Fixes #23600.

Also remove another !important -- and there was much rejoicing.

Props SergeyBiryukov and obenland.

Note: See TracTickets for help on using tickets.