Skip to:

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#5061 closed enhancement (fixed)

Add confirmation to "leave group" button

Reported by: dcavins's profile dcavins Owned by: boonebgorges's profile boonebgorges
Milestone: 1.9 Priority: low
Severity: minor Version:
Component: Groups Keywords: has-patch 2nd-opinion


My users have requested that it not be quite so easy to leave a group, especially by accident. I've added a confirmation step, hopefully saving the user hassles when the group is private or hidden and he hit the button by mistake. :)

Attachments (1)

leave-group-confirmation.patch (3.5 KB) - added by dcavins 11 years ago.
Adds confirmation dialog to leave-group action

Download all attachments as: .zip

Change History (5)

11 years ago

Adds confirmation dialog to leave-group action

#1 @boonebgorges
11 years ago

  • Keywords 2nd-opinion added
  • Milestone changed from Awaiting Review to 1.9
  • Priority changed from normal to low

Moving to 1.9 for feedback from other devs. I don't have a problem with it personally.

#2 follow-up: @boonebgorges
11 years ago

I think I'm going to leave this enhancement out of bp-default.

#3 @boonebgorges
11 years ago

  • Owner set to boonebgorges
  • Resolution set to fixed
  • Status changed from new to closed

In 7458:

Add confirmation for Leave Group buttons in bp-legacy templates.

Fixes #5061

Props dcavins

#4 in reply to: ↑ 2 @dcavins
11 years ago

Replying to boonebgorges:

I think I'm going to leave this enhancement out of bp-default.

Seems reasonable, given the future of bp-default. It does make me realize that I need to stop relying on bp-default in my custom theme and move over to the modern theme compatibility route myself.

Note: See TracTickets for help on using tickets.