Skip to:
Content

bbPress.org

Opened 15 years ago

Closed 15 years ago

#1691 closed enhancement (fixed)

current_theme_supports() is confusing

Reported by: johnjamesjacoby Owned by:
Priority: normal Milestone: 2.1
Component: Appearance - Theme Compatibility Version: 2.0
Severity: normal 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
15 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
15 years ago

  • Resolutionfixed
  • Status newclosed

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.