Ticket #1755 (closed enhancement: fixed)

Opened 3 months ago

Last modified 3 months ago

Make the bpp-twentyten content wrapper areas hookable

Reported by: aprea 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

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from Awaiting Review to 2.1

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!

Note: See TracTickets for help on using tickets.