Skip to:
Content

bbPress.org

Opened 9 years ago

Last modified 6 years ago

#1298 new enhancement

Better post author layout management

Reported by: Nightgunner5 Owned by:
Milestone: Future Release (Legacy) Priority: normal
Severity: normal Version:
Component: General - Administration Keywords: has-patch tested
Cc: pippin@…

Description

With so many plugins trying to override each other and change post_author_title to insert things that aren’t the post author’s title, it just makes sense to let users choose where everything goes.

It's also easier on forum admins that don't know how to use PHP or don't want to edit their theme's files, but want to change the layout of the post author box.

It's simple for a plugin or theme to work both ways:

if ( function_exists( 'bb_post_author_layout' ) ) {
    // Old logic, same as always.
} else {
    // THEMES:
    bb_post_author_layout();

    // PLUGINS:
    function my_plugin_section_foo() {
        echo 'Foo';
    }

    function my_plugin_section_bar() {
        echo 'Bar';
    }

    function my_plugin_post_author_sections_add( $sections ) {
        $sections[] = 'my_plugin_section_foo';
        $sections[] = 'my_plugin_section_bar';
        return $sections;
    }
    add_filter( 'bb_post_author_layout_sections', 'my_plugin_post_author_sections_add' );
}

Attachments (3)

post-author-layout.diff (6.4 KB) - added by Nightgunner5 9 years ago.
post-author-layout.png (67.6 KB) - added by Nightgunner5 9 years ago.
post-author-layout-2.diff (6.8 KB) - added by Nightgunner5 9 years ago.
Allow plugins to return false, making the before and after not take up space.

Download all attachments as: .zip

Change History (8)

#1 @Nightgunner5
9 years ago

Additionally, plugins that only use the old method will still work, but they won't be separate from the author title block (assuming they use that one).

For plugins like Post Count Plus that remove the title, this simply turns the title block into a Post Count Plus block.

@Nightgunner5
9 years ago

Allow plugins to return false, making the before and after not take up space.

#2 @Nightgunner5
9 years ago

Oops - the example is backwards. There should be a ! before function_exists

#3 @johnjamesjacoby
8 years ago

  • Keywords changed from has-patch, tested to has-patch tested
  • Milestone changed from Future Release to 1.1
  • Version 1.0.1 deleted

!Nightgunner5 - Still care to clean this up?

#4 @johnjamesjacoby
8 years ago

  • Milestone changed from 1.1 to 1.2

#5 @mordauk
6 years ago

  • Cc pippin@… added

Do we still want to do this? I'll take over if so.

Note: See TracTickets for help on using tickets.