[buddypress-trac] [BuddyPress] #4089: add_theme_support BP-Default Theme

buddypress-trac at lists.automattic.com buddypress-trac at lists.automattic.com
Tue Mar 20 10:38:51 UTC 2012


#4089: add_theme_support BP-Default Theme
--------------------------+------------------------------
 Reporter:  chrisclayton  |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Theme         |     Version:
 Severity:  normal        |  Resolution:
 Keywords:                |
--------------------------+------------------------------

Comment (by chrisclayton):

 We adopted use of current_theme_supports with commits r5747 and r5749

 I was pretty much thinking the same thing you mentioned above, but pretty
 much wrapping most of that using current theme support.

 I was going to write a patch when i posted this, but i thought i'd get
 some opinions if it's worth writing it, I personally am i huge fan
 child/parent themes due to their upgradeability and thus i'm really
 passionate about fixing whatever is stopping alot of of these 'feature
 rich' themes building them as parent themes, especially a theme being
 marked as a bp core project (#3662), as i'm constantly reminded of
 WordPress' philosophy of 'Design for the majority'
 http://wordpress.org/about/philosophy/ (which i always thought was BP's
 philosophy too) But; i'm not sure if this will even fix those issues.

 "I'm not sure what it would mean in the case of bp-default itself."

 Since bp-default is packaged within the plugin, i thought it would be
 pretty much the same as adding any other feature with add_theme_support,
 since the theme doesn't get touched (as if it doesn't exist) if it's not
 active the code would be in core, but i classified this ticket it in the
 'theme' component since that is what it's directly related too :)

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4089#comment:3>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list