[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