[wp-trac] [WordPress Trac] #20448: Update Twenty Ten and Twenty Eleven to use 3.4 features
WordPress Trac
wp-trac at lists.automattic.com
Tue Jun 5 18:08:48 UTC 2012
#20448: Update Twenty Ten and Twenty Eleven to use 3.4 features
----------------------------+--------------------------
Reporter: nacin | Owner: koopersmith
Type: task (blessed) | Status: reopened
Priority: high | Milestone: 3.4
Component: Bundled Theme | Version: 3.4
Severity: major | Resolution:
Keywords: has-patch |
----------------------------+--------------------------
Comment (by nacin):
Okay, let's bury the idea that, if un-saved defaults are in place, there
should be no <style> block printed. This hurts themes that used
BACKGROUND_IMAGE without specifying a fallback in the stylesheet, and they
did that because otherwise removal wouldn't work.
[attachment:20448.8.diff] reverts wp-includes for [20973] and then makes
some adjustments:
* <style> will appear if there is a default image registered. This is the
same as 3.3.
* If only a default color is registered, it still assumes it is in the
stylesheet, and no <style> will appear. This is a change from 3.3.
* <style> will continue to appear as before if there is a custom
background color or image. This is the same as 3.3.
* This then allows for a default background image with background-image:
none, overriding style.css. This is new.
The goals of [20973] were to make default-image and default-color not
suck, thus encouraging their use, since they're basically new in 3.4.* But
at the same time we don't want to encourage people to stop putting these
things in CSS '''as well'''. We simply want them to register it with
WordPress so we know.
(*) I consider them new because:
* They are now first-class arguments, rather than second-class constants.
People will notice them.
* The constants did not work well, as they could not be removed, were not
part of the preview, and were not represented in the UI.
* The default themes are now leveraging them.
This does not fix comment 15.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/20448#comment:19>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list