Opened 13 years ago
Closed 13 years ago
#1755 closed enhancement (fixed)
Make the bpp-twentyten content wrapper areas hookable
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 2.1 | Priority: | normal |
Severity: | normal | Version: | 2.0 |
Component: | API - Actions/Filters | Keywords: | |
Cc: |
Description
At the moment the bbp-twentyten theme has it's content wrappers hardcoded into the template files, i.e.
<div id="container"> <div id="content" role="main"> <!-- template code here --> </div><!-- #content --> </div><!-- #container -->
I think a more intuitive approach would be something like this:
<?php do_action( 'bbp_before_main_content' ); ?> <!-- template code here --> <?php do_action( 'bbp_after_main_content' ); ?>
That way it's MUCH easier to theme bbPress for child / parent themes.
Idea taken from jigoshop, they explain it pretty well here: http://forum.jigoshop.com/kb/shortcodes/theming-with-jigoshop
Change History (1)
Note: See
TracTickets for help on using
tickets.
This is fixed in the new bbp-theme-compat folder. There's more to do in there, but it's unrelated to this ticket.
Closing as fixed. Thanks for the suggestion!