Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 8 years ago

#3297 closed defect (bug) (no action required)

bp_before_directory_groups_content hook

Reported by: tarion's profile 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() ) : ?> &nbsp;<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)

#1 @DJPaul
13 years ago

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

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

#2 @johnjamesjacoby
13 years ago

  • Milestone changed from Awaiting Review to 1.5
  • Severity set to normal
  • Version 1.2.8 deleted

Moving closed ticket out of Awaiting Review.

#3 @DJPaul
8 years ago

  • Component changed from Appearance - Template Parts to Templates
Note: See TracTickets for help on using tickets.