Skip to:
Content

BuddyPress.org

Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#8497 closed enhancement (fixed)

Allow detecting button element with bp_nouveau_get_members_buttons and bp_nouveau_get_groups_buttons

Reported by: sbrajesh Owned by: imath
Priority: normal Milestone: 9.0.0
Component: Templates Version: 3.0.0
Severity: normal Keywords: has-patch commit
Cc:

Description

Hi,
Currently, if 3rd party code wants to add extra buttons using the above 2 filters, they do not know if they should use button or anchor element.
These filters should pass the user passed $args to allow the detection of button element being used.

Please find the patch atatched.

Attachments (1)

8497.diff​ (1.7 KB ) - added by sbrajesh 5 years ago.

Download all attachments as: .zip

Change History (4)

@sbrajesh
5 years ago

#1 @imath
5 years ago

  • Component (not sure) → Templates
  • Keywords commit added
  • Milestone Awaiting Review → 9.0.0
  • Version 8.0.0 → 3.0.0

Thanks @sbrajesh for your suggestion & patch. It looks good 👍

#2 @imath
5 years ago

  • Owner set to imath
  • Resolution → fixed
  • Status new → closed

In 12979:

Pass expected arguments to Members/Groups Action button filters

To improve extensibility of the 2 functions BP Nouveau uses to generate these action buttons (bp_nouveau_get_members_buttons() & bp_nouveau_get_groups_buttons()), we are adding a new $args parameter to their corresponding filters ('bp_nouveau_get_members_buttons'& 'bp_nouveau_get_groups_buttons').

Doing so should help developers to have better information about the expected output for these buttons.

Props sbrajesh

Fixes #8497

#3 @sbrajesh
5 years ago

Thank you.

Note: See TracTickets for help on using tickets.