#5077 Add confirmation when removing developer

v1.0.0
closed
nobody
General
2015-08-20
2012-10-05
Chris Tsai
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

    Dave Brondsema - 2012-10-05
    • milestone: limbo --> forge-nov-02
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-16
    • labels: support, p3 --> support, p3, 42cc
     
  • Igor Bondarenko - 2012-10-17

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

     

    Related

    Tickets: #5077

  • Igor Bondarenko - 2012-10-17
    • status: open --> in-progress
     
  • Igor Bondarenko - 2012-10-18

    Closed #191. Branch 42cc_5077.

     
  • Igor Bondarenko - 2012-10-18
    • status: in-progress --> code-review
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-22
    • status: code-review --> closed
    • qa: Dave Brondsema
    • size: 1 --> 0
    • milestone: forge-nov-16 --> forge-nov-02
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-26
    • status: closed --> open
    • qa: Dave Brondsema --> nobody
    • milestone: forge-nov-02 --> forge-backlog
     
  • Dave Brondsema

    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.

     
  • Igor Bondarenko - 2012-10-29

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

     

    Related

    Tickets: #5077

  • Igor Bondarenko - 2012-10-29
    • status: open --> in-progress
     
  • Igor Bondarenko - 2012-10-30
    • status: in-progress --> code-review
     
  • 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{
    
     
  • Dave Brondsema

    Dave Brondsema - 2012-10-30
    • status: code-review --> closed
    • qa: Dave Brondsema
    • milestone: forge-backlog --> forge-nov-02
     

Log in to post a comment.