Skip to:
Content

bbPress.org

Opened 13 years ago

Closed 13 years ago

#1755 closed enhancement (fixed)

Make the bpp-twentyten content wrapper areas hookable

Reported by: aprea's profile aprea 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)

#1 @johnjamesjacoby
13 years ago

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

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.