Apache Allura™

Forge software for hosting software projects

Brought to you by: alexluberg, brondsem, ccruz, deshani, and 11 others

#5077 Add confirmation when removing developer

Milestone: v1.0.0

Status: closed

Owner: nobody

Labels: support (446) p3 (267) 42cc (432)

Component: General

Reviewer: Dave Brondsema

Updated: 2015-08-20

Created: 2012-10-05

Creator: Chris Tsai

Private: No

Users can too easily remove themselves (or other admins) since it just takes a single click, I've personally needed to re-add a user after they accidentally removed themselves. We should add a confirmation.

Also reducing the click target to just the "-" for removing would help reduce the incidence of this.

Related

Tickets: ~~#5077~~
Tickets: ~~#5189~~

Discussion

Dave Brondsema - 2012-10-05

milestone: limbo --> forge-nov-02
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2012-10-16

labels: support, p3 --> support, p3, 42cc
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-17

Created #191: [#5077] Add confirmation when removing developer (1cp)

Related

Tickets: ~~#5077~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-17

status: open --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-18

Closed #191. Branch 42cc_5077.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-18

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2012-10-22

status: code-review --> closed

qa: Dave Brondsema

size: 1 --> 0

milestone: forge-nov-16 --> forge-nov-02
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2012-10-26

status: closed --> open

qa: Dave Brondsema --> nobody

milestone: forge-nov-02 --> forge-backlog
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2012-10-26

Hi, we've merged the changes here but actually would like one more change. It would be nice if pointer didn't change when mousing over the username. That made sense with the old behavior, but it doesn't now that the click target is just the icon.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-29

Created #199: [#5077] Do not change mouse pointer over username (1cp)

Related

Tickets: ~~#5077~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-29

status: open --> in-progress
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-30

status: in-progress --> code-review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Igor Bondarenko - 2012-10-30

Closed #199.

We don't have push access to the sftheme repo, so I'm just leaving diff here since it's trivial change to sourceforge theme.

--- a/allura/sftheme/nf/sftheme/sass/partials/_admin.scss +++ b/allura/sftheme/nf/sftheme/sass/partials/_admin.scss @@ -34,7 +34,7 @@ } } } - li.deleter { + b.deleter { cursor: pointer; } .parent_users{
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dave Brondsema - 2012-10-30

status: code-review --> closed

qa: Dave Brondsema

milestone: forge-backlog --> forge-nov-02
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.