#3297 closed defect (bug) (no action required)
bp_before_directory_groups_content hook
Reported by: | tarion | Owned by: | |
---|---|---|---|
Milestone: | 1.5 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Templates | Keywords: | |
Cc: |
Description
Thats how it is:
<h3><?php _e( 'Groups Directory', 'buddypress' ) ?><?php if ( is_user_logged_in() ) : ?> <a class="button" href="<?php echo bp_get_root_domain() . '/' . BP_GROUPS_SLUG . '/create/' ?>"><?php _e( 'Create a Group', 'buddypress' ) ?></a><?php endif; ?></h3> <?php do_action( 'bp_before_directory_groups_content' ) ?> <div id="group-dir-search" class="dir-search"> <?php bp_directory_groups_search_form() ?> </div><!-- #group-dir-search -->
But the do_action should be after the search box. Because the default layout has the searchbox right floating of the the heading. Adding content in between will destroy the layout.
File:
bp-themes\bp-default\groups\index.php
Change History (3)
Note: See
TracTickets for help on using
tickets.
In trunk (1.3), we've moved the 'bp_before_directory_groups_content' action to the top of the file to normalise with the other templates. I think that change will "fix" the issue in this ticket.
See https://buddypress.trac.wordpress.org/browser/trunk/bp-themes/bp-default/groups/index.php#L14