[wp-trac] [WordPress Trac] #25540: Twenty Fourteen: Remove Header Text Color Customization Option
WordPress Trac
noreply at wordpress.org
Fri Oct 11 08:53:00 UTC 2013
#25540: Twenty Fourteen: Remove Header Text Color Customization Option
------------------------------+------------------
Reporter: celloexpressions | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.8
Component: Bundled Theme | Version:
Severity: normal | Resolution:
Keywords: 2nd-opinion |
------------------------------+------------------
Comment (by celloexpressions):
After digging around in core, it looks like adding support for not showing
the color option is tricky. The whole custom header/custom background
implementation is... messy. (For the record, I was thinking we could add
the ability to disable the text color by setting the default to `'none'`,
or something along those lines. But then we'd also need to deal with back-
compatibility in the theme, and the alternate solution is much more
elegant.)
However, given the UI mess surrounding the custom header page, I don't
think there's much benefit to worrying about hiding the color option
there. Hopefully we can ditch that whole screen soon anyway (#25571).
Where it really matters, especially if the separate page is eventually
gone, it the customizer. And that's where having an almost useless option
next to (and before) two really powerful options is particularly
offensive.
'''I think our best option is actually to just remove the control from the
customizer'''. Since we're already modifying other existing controls, this
is actually completely acceptable as the recommended/default-theme way to
hide unneeded controls from the customizer. After all, this is kind of the
point of having the `remove_control` function. (As an aside, this would
also further showcase the customizer in Twenty Fourteen since we'd
demonstrate adding controls, removing them, and modifying existing ones.)
I'm adding a patch that removes the control from the customizer. I
honestly don't care about the Custom Header page, and don't think many
users spend much time there, especially if they didn't use WordPress much
before 3.4 (ie, newer users are increasingly more likely to use the
customizer as much as possible as it becomes more prominent).
If we do decide to keep this control around, we should at least add
postMessage support to it, like the background color has (though this
isn't as realistic for the accent color, unfortunately).
--
Ticket URL: <http://core.trac.wordpress.org/ticket/25540#comment:6>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list