[wp-trac] [WordPress Trac] #25094: Twenty Fourteen: Audit Theme Customization Options
WordPress Trac
noreply at wordpress.org
Sun Sep 1 08:32:30 UTC 2013
#25094: Twenty Fourteen: Audit Theme Customization Options
------------------------------+--------------------
Reporter: celloexpressions | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.8
Component: Bundled Theme | Version: trunk
Severity: normal | Resolution:
Keywords: has-patch |
------------------------------+--------------------
Comment (by celloexpressions):
[attachment:25094.accent-color.2.diff] implements obenland's suggestions
and also only outputs the color overrides if a custom color is selected
(borrowed from Twenty Eleven); this way, child themes and custom css
aren't overridden. We can also keep the current shades for all three
default colors, since they aren't generating.
I think Accent Color is better because we're changing more than just the
links. This control lets you change ''all'' of the colors in Twenty
Fourteen other than the background and grayscale tones. While that isn't
much beyond the links (notably, it covers the header search button and the
text selection color), I think the visual impact warrants a more
substantial label.
In terms of the internal variable names, both generated colors are
actually lighter than the original, so I'm not sure what the best naming
convention would be.
I would like to apply the primary accent (or whatever we call it) color to
the editor stylesheet for links and ::selection but the only method I know
of to do that requires creating a file; anyone know a better way to load a
few lines of css into TinyMCE?
Revisiting the color generation itself, anyone else think we should
lighten the first generated color a bit more than 14 steps? The best place
to test the contrast between those shades is the normal and hover states
of the search button; I've found the contrast to be small enough to look
unintentional for several initial colors.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/25094#comment:19>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list