# Auto-add permissions/unsaved permissions prompt

**URL:** https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919
**Category:** UX
**Created:** [10월 30, 2018, 9:07오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919 "2018-10-30T21:07:00Z")
**Posts on this page:** 13
**Page:** 1

<div class="post-metadata">

### Author: ![Mark\_Walkom](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/mark_walkom/32/77411_2.png) [@Mark\_Walkom](https://meta.discourse.org/u/Mark_Walkom)
#### Post date: [10월 30, 2018, 9:07오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/1 "2018-10-30T21:07:00Z")

</div>

When editing permissions, it can be easy to add a new set but forget to click the `Add Permission` to apply it before closing the edit box (don’t judge me! :p).

Would it be possible to have either a reminder - saying `Unapplied Permissions` or similar - or just apply the new permissions when clicking `Save Category`?

---

<div class="post-metadata">

### Author: ![Falco](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/falco/32/179432_2.png) [@Falco](https://meta.discourse.org/u/Falco)
#### Post date: [11월 1, 2018, 9:45오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/2 "2018-11-01T21:45:32Z")

</div>

I second this. I did forget the apply too many times!

---

<div class="post-metadata">

### Author: ![simon](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/simon/32/339122_2.png) [@simon](https://meta.discourse.org/u/simon)
#### Post date: [11월 1, 2018, 10:00오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/3 "2018-11-01T22:00:45Z")

</div>

I have forgotten to to this as well, but making a mistake by adding the wrong group to category permissions could have serious consequences. I think having requiring confirmation here is a good idea.

Without a confirmation, there would need to be some way to dismiss a permission after the Edit Permissions button has been clicked.

 ![44%20PM](https://global.discourse-cdn.com/meta/original/3X/e/3/e3b058c570ce1dc4d9ff4c9daac3cbe1693b0c05.png)

If it needs to be changed, maybe the Add Permission button could be changed to a green checkmark box, similar to what is used in the Setting controls.

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [11월 2, 2018, 6:15오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/4 "2018-11-02T18:15:45Z")

</div>

Maybe a good starter task @erlend_sh?

---

<div class="post-metadata">

### Author: ![mipe](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/mipe/32/129745_2.png) [@mipe](https://meta.discourse.org/u/mipe)
#### Post date: [2월 12, 2019, 8:36오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/5 "2019-02-12T08:36:25Z")

</div>

Hello,

I wanted to contribute and looked at this issue, I think it would be good enough if the design could be changed: By renaming the “Add Permission” Button into just “Add” and moving it to the same line as the combo-boxes:

 ![Screenshot%20from%202019-02-12%2017-33-09](https://global.discourse-cdn.com/meta/original/3X/5/0/5045dc3f7ee69856f39c0963bad10492bddcbb9d.png)  
It would be intuitive to click, rather than changing the logic (which would involve checking if something has changed in the combo-boxes and then showing a Message).  
Let me know what you think

---

<div class="post-metadata">

### Author: ![mipe](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/mipe/32/129745_2.png) [@mipe](https://meta.discourse.org/u/mipe)
#### Post date: [2월 13, 2019, 2:40오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/6 "2019-02-13T02:40:12Z")

</div>

Hello, I made the change and created a PR:  
[https://github.com/discourse/discourse/pull/6999](https://github.com/discourse/discourse/pull/6999)

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [2월 14, 2019, 4:37오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/7 "2019-02-14T04:37:49Z")

</div>

I worry that this will not fit in some non English locales. maybe if its in some magic flexbox it would work. @awesomerobot can you have a very quick look at this.

---

<div class="post-metadata">

### Author: ![mipe](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/mipe/32/129745_2.png) [@mipe](https://meta.discourse.org/u/mipe)
#### Post date: [2월 14, 2019, 4:45오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/8 "2019-02-14T04:45:54Z")

</div>

Yes, I figured I change it to a “+” instead of “Add”, sorry I haven’t mentioned that in the previous post

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [2월 14, 2019, 5:43오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/9 "2019-02-14T05:43:10Z")

</div>

I can see:

```plaintext
    margin-left: 0.25em;
    position: absolute;

```

so there is still a real chance this thing will overlap, we should probably use a flexbox here or float the element right or something.

---

<div class="post-metadata">

### Author: ![awesomerobot](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/awesomerobot/32/142900_2.png) [@awesomerobot](https://meta.discourse.org/u/awesomerobot)
#### Post date: [2월 14, 2019, 3:41오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/11 "2019-02-14T15:41:18Z")

</div>

[Commented inline on Github](https://github.com/discourse/discourse/pull/6999/files#r256887046), but that’s the gist — `position: absolute` is unneeded here. Everything else seems ok with long strings.

> [@mipe](#):
>
> Yes, I figured I change it to a “+” instead of “Add”, sorry I haven’t mentioned that in the previous post

Good call, removing text certainly makes localization easier.

---

<div class="post-metadata">

### Author: ![mipe](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/mipe/32/129745_2.png) [@mipe](https://meta.discourse.org/u/mipe)
#### Post date: [2월 15, 2019, 3:04오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/12 "2019-02-15T03:04:44Z")

</div>

Thanks a lot for the review, I changed it accordingly and added a right margin to the `permission-selector`.  
In order to make the add-button align with the combo-boxes, I did the following:  
` .add-permission { position: relative; top: 0.1em; }`  
otherwise the button would float a few pixels higher then the combo-boxes.  
Open for suggestions if there’s a better way to do it 🙂

---

<div class="post-metadata">

### Author: ![awesomerobot](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/awesomerobot/32/142900_2.png) [@awesomerobot](https://meta.discourse.org/u/awesomerobot)
#### Post date: [2월 15, 2019, 5:03오후 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/13 "2019-02-15T17:03:05Z")

</div>

Great, I just merged in the change [UX: make adding permissions more visible and intuitive to click. (#6999) · discourse/discourse@159dd69 · GitHub](https://github.com/discourse/discourse/commit/159dd69fcb5c687fe200ba27b5753df0e85834c2). The button alignment is fine for now, there’s a bigger fix we’ll work on that should better resolve the alignment issues.

---

<div class="post-metadata">

### Author: ![sam](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/sam/32/102149_2.png) [@sam](https://meta.discourse.org/u/sam)
#### Post date: [2월 18, 2019, 3:32오전 UTC](https://meta.discourse.org/t/auto-add-permissions-unsaved-permissions-prompt/100919/14 "2019-02-18T03:32:43Z")

</div>


