[UX] Enable Custom Permissions : checkbox should be mandatory to be able to change roles/actions sheet #1085

Closed
opened 2026-02-04 23:42:44 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @tpoindessous on GitHub (Mar 11, 2019).

Hi !

I made a shelve and I wanted to limit permission to a single role. So I checked the right boxes for this role. I saved my changes and still, users with this role couldn't see shelve. It takes me a long time to figure that I needed to check "Enable Custom Permissions"

Describe the feature you'd like
If "Enable Custom Permissions" is not checked, I can't check boxes in roles/actions sheet.
The documentation should also mention that.

Describe the benefits this feature would bring to BookStack users
It would be more explicit that this checkbox is mandatory for editing custom permissions.

Thanks !

Originally created by @tpoindessous on GitHub (Mar 11, 2019). Hi ! I made a shelve and I wanted to limit permission to a single role. So I checked the right boxes for this role. I saved my changes and still, users with this role couldn't see shelve. It takes me a long time to figure that I needed to check "Enable Custom Permissions" **Describe the feature you'd like** If "Enable Custom Permissions" is not checked, I can't check boxes in roles/actions sheet. The documentation should also mention that. **Describe the benefits this feature would bring to BookStack users** It would be more explicit that this checkbox is mandatory for editing custom permissions. Thanks !
OVERLORD added the 💆 UX label 2026-02-04 23:42:44 +03:00
Author
Owner

@ssddanbrown commented on GitHub (Mar 13, 2019):

Thanks for your feedback @tpoindessous.

This is somewhat intentional, To allow a user to easily toggle custom permissions on an entity while still saving the role options in the event permissions are re-inforced.

That said, I do admit this is bad UX.
A bigger revamp is due for the permission system later in the road map. It will likely be at that point this will be improved upon.

@ssddanbrown commented on GitHub (Mar 13, 2019): Thanks for your feedback @tpoindessous. This is somewhat intentional, To allow a user to easily toggle custom permissions on an entity while still saving the role options in the event permissions are re-inforced. That said, I do admit this is bad UX. A bigger revamp is due for the permission system later in the [road map](https://github.com/BookStackApp/BookStack#road-map). It will likely be at that point this will be improved upon.
Author
Owner

@ssddanbrown commented on GitHub (Sep 4, 2019):

Just came across this again, @timoschwarzer effectively covered this issue in the changes within #1505, Which hides the permission checkbox table unless custom permissions are active. This change was included in v0.27.

Therefore I'll close this off.

@ssddanbrown commented on GitHub (Sep 4, 2019): Just came across this again, @timoschwarzer effectively covered this issue in the changes within #1505, Which hides the permission checkbox table unless custom permissions are active. This change was included in v0.27. Therefore I'll close this off.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/BookStack#1085