[wp-trac] [WordPress Trac] #27863: Move $content_width inside setup function in core-bundled Themes

WordPress Trac noreply at wordpress.org
Fri May 1 13:55:12 UTC 2015


#27863: Move $content_width inside setup function in core-bundled Themes
------------------------------------------+-----------------------------
 Reporter:  chipbennett                   |       Owner:
     Type:  enhancement                   |      Status:  new
 Priority:  normal                        |   Milestone:  Future Release
Component:  Bundled Theme                 |     Version:  3.9
 Severity:  normal                        |  Resolution:
 Keywords:  has-patch dev-feedback close  |     Focuses:
------------------------------------------+-----------------------------

Comment (by chipbennett):

 The biggest issue for me personally, as part of the Theme Review Team, is
 that the core-bundled Themes (and Underscores) are the most-used Themes
 for derivative works. Thus, when the core-bundled Themes do something one
 way, that method gets propagated, readily.

 I think there is inherent benefit in continually raising the bar, to
 promote best practices - especially when the practices in question are
 known to be learned and used by a very wide audience.

 While making this change in the next core-bundled Theme will be a good
 start, the problem is that the existing core-bundled Themes remain active,
 and actively used.

 I admittedly have another, selfish motive: it is tedious continually
 having to explain to Theme developers during Theme reviews that, yes, they
 are required to follow a Theme review guideline, even though the core-
 bundled Theme from which they derived their own Theme doesn't follow that
 guideline - and that core-bundled Themes won't necessarily adhere to all
 guidelines, because the Theme Review Team has no direct control over core-
 bundled Themes.

 And also, if there's ever any movement on #21256, moving `$content_width`
 inside a callback will make for a much easier transition.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/27863#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list