Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 15 years ago

Last modified 10 years ago

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

bp_before_directory_groups_content hook

Reported by: tarion Owned by:
Priority: normal Milestone: 1.5
Component: Templates Version:
Severity: normal 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
15 years ago

  • Resolutioninvalid
  • Status newclosed

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
15 years ago

  • Milestone Awaiting Review1.5
  • Severitynormal
  • Version 1.2.8

Moving closed ticket out of Awaiting Review.

#3 @DJPaul
10 years ago

  • Component Appearance - Template PartsTemplates
Note: See TracTickets for help on using tickets.