Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Security admin shows confusing permissions for super admin #3310

Closed
stojg opened this issue Jul 20, 2014 · 4 comments
Closed

Security admin shows confusing permissions for super admin #3310

stojg opened this issue Jul 20, 2014 · 4 comments

Comments

@stojg
Copy link

stojg commented Jul 20, 2014

I believe it would be better if we can have all boxes under the administrator green or at least greyed out when the user is a super user.

@clarkepaul, do you have any opinions / preferences?

This is the view when looking at the Members permissions
screen shot 2014-03-28 at 3 33 25 pm

It' looks better when looking at the permissions tab in the groups

I noticed that the group permissions shows the permissions as greyed out:

screenshot 2014-07-18 11 05 41

Ticket moved from silverstripe/silverstripe-cms#985

@clarkepaul clarkepaul added this to the CMS 4.0.0-beta1 milestone Sep 6, 2016
@clarkepaul
Copy link
Contributor

clarkepaul commented Sep 6, 2016

Hey one year later reply, yes I agree! Moved up the priority of this.

@clarkepaul
Copy link
Contributor

Related #4861

@zacps
Copy link
Contributor

zacps commented Jan 17, 2017

I'm working on this now. Any opinions on whether they should be green or disabled? I feel like disabled makes more sense because it makes it clear that none of the other permissions are relevant when you're a superuser.

@stojg
Copy link
Author

stojg commented Jan 17, 2017

Disabled sounds good to me since the check boxes are disabled as well.

zacps added a commit to zacps/silverstripe-framework that referenced this issue Jan 18, 2017
This changes permissions inherited by being a superuser to gray,
instead of the red x that made it look like you didn't have it.
zacps added a commit to zacps/silverstripe-framework that referenced this issue Jan 18, 2017
This changes permissions inherited by being a superuser to gray,
instead of the red x that made it look like you didn't have it.
dhensby added a commit that referenced this issue Jan 27, 2017
Simplify permissions displayed when admin (fixes #3310)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants