#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

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

    Igor Bondarenko - 2012-10-17

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

     

    Related

    Tickets: #5077

  • Igor Bondarenko

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

    Igor Bondarenko - 2012-10-18

    Closed #191. Branch 42cc_5077.

     
  • Igor Bondarenko

    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

    Igor Bondarenko - 2012-10-29

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

     

    Related

    Tickets: #5077

  • Igor Bondarenko

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

    Igor Bondarenko - 2012-10-30
    • status: in-progress --> code-review
     
  • Igor Bondarenko

    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.