[wp-trac] [WordPress Trac] #26233: Twenty Fourteen: prefix the Featured_Content class and options

WordPress Trac noreply at wordpress.org
Tue Nov 26 08:14:55 UTC 2013


#26233: Twenty Fourteen: prefix the Featured_Content class and options
-----------------------------+--------------------
 Reporter:  kovshenin        |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  3.8
Component:  Bundled Theme    |     Version:  trunk
 Severity:  normal           |  Resolution:
 Keywords:  has-patch close  |
-----------------------------+--------------------
Changes (by obenland):

 * keywords:  has-patch => has-patch close


Comment:

 The class should not be in Twenty Fourteen in the first place - the theme
 should only have to care about the display of featured content, not how
 it's managed. Bringing it in was a compromise to enable support for a
 slider that is smarter than using sticky posts. Using post tags is far
 from ideal, but it's the next best thing.

 This is not an oversight. The reason to leave it unprefixed is to make it
 pluggable for existing/future featured content solutions, like the afore
 mentioned core plugin. It will also make Twenty Fourteen forward
 compatible if/when a FC solution gets into core.

 Let's keep it as is and make it as easy as possible for plugins to enhance
 the functionality.

--
Ticket URL: <http://core.trac.wordpress.org/ticket/26233#comment:5>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list