#5077 Add confirmation when removing developer

asf_release_1.0.0
closed
nobody
General
2014-11-23
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
     
  • Created #191: [#5077] Add confirmation when removing developer (1cp)

     

    Related

    Tickets: #5077

    • status: open --> in-progress
     
  • Closed #191. Branch 42cc_5077.

     
    • 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.

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

     

    Related

    Tickets: #5077

    • status: open --> in-progress
     
    • status: in-progress --> code-review
     
  • 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