Ticket #1755 (closed enhancement: fixed)
Make the bpp-twentyten content wrapper areas hookable
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 2.1 |
| Component: | Actions/Filters | Version: | 2.0 |
| Severity: | normal | 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
comment:1
johnjamesjacoby — 3 months ago
- Status changed from new to closed
- Resolution set to fixed
- Milestone changed from Awaiting Review to 2.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!