-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
ui(rule): hide inventory rules menu items when user lacks view rights #22988
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
base: 11.0/bugfixes
Are you sure you want to change the base?
Conversation
|
If the user has no permissions on rules (globally), should GLPI display the ‘Rules’ menu at all? |
|
I'm not sure for the view. |
And probably, yes |
trasher
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comments
|
Agree with @trasher |
|
Moreover, if I enable the permission on software dictionaries or on business rules for tickets, the tile remains unclickable and red (I should be able to access it). |
Rom1-B
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For the visual, red is indeed misleading, so we could possibly slightly gray out what is not accessible and change the cursor:
cursor: not-allowed;
stonebuzz
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to add a tooltip on hover over the tile indicating “You do not have the required permissions”?

Description
Screenshots (if appropriate):
Rights:

View
