Skip to:
Content

bbPress.org

Opened 14 years ago

Closed 13 years ago

#1691 closed enhancement (fixed)

current_theme_supports() is confusing

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by:
Milestone: 2.1 Priority: normal
Severity: normal Version: 2.0
Component: Appearance - Theme Compatibility Keywords:
Cc:

Description

v1 of theme compatibility worked well but developers have difficulty understanding how it works and how to extend it. Also, making a new theme pack is harder than it should be.

For v2, I propose we remove the current_theme_supports( 'bbpress' ) requirement, and look for template files in the most logical places.

  • Parent Theme
  • Child Theme
  • Theme Compat Directory

This will require a bbp_locate_template() function to mirror the WordPress core locate_template() function, but includes the theme compat folder too.

Change History (2)

#1 @johnjamesjacoby
14 years ago

(In [3623]) First pass at v2 theme compatibility:

  • Remove references to current_theme_supports()
  • Introduce bbp_locate_template() to include theme compat path
  • Use bbp_locate_template() through-out project
  • See #1691

#2 @johnjamesjacoby
13 years ago

  • Resolution set to fixed
  • Status changed from new to closed

Closing as fixed. All references have been removed except for where shortcode override may not be warranted.

Note: See TracTickets for help on using tickets.