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

[Enhancement]: Improve UX for DNN 10 Permissions grid #6175

Open
2 of 4 tasks
Timo-Breumelhof opened this issue Oct 25, 2024 · 3 comments
Open
2 of 4 tasks

[Enhancement]: Improve UX for DNN 10 Permissions grid #6175

Timo-Breumelhof opened this issue Oct 25, 2024 · 3 comments

Comments

@Timo-Breumelhof
Copy link
Contributor

Timo-Breumelhof commented Oct 25, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Description of problem

The addition of the more granular edit permission settings to DNN 10 is great.
But IMO the current UX is not ideal.

See:

image

I see a few issues:

  1. Edit is at the end although in most use cases view and edit are the most important
  2. add - navigate are essentially sub categories of Edit, which is not clear until you click edit
  3. Because a lot of the texts are not visible understanding what does what is quite difficult.

Description of solution

  1. Position Edit just after view
  2. Is possible add a separator between edit and the rest of the items to show the difference between the two

See:
image

  1. We could also replace the texts of the "advanced" items to an icon, which is clearer in narrow space.

Example:

image

Description of alternatives considered

up for discussion :-)

Anything else?

No response

Do you be plan to contribute code for this enhancement?

  • Yes

Would you be interested in sponsoring this enhancement?

  • Yes

Code of Conduct

  • I agree to follow this project's Code of Conduct
@Timo-Breumelhof Timo-Breumelhof changed the title [Enhancement]: Improve UX for Permissions grid [Enhancement]: Improve UX for DNN 10 Permissions grid Oct 25, 2024
@Timo-Breumelhof
Copy link
Contributor Author

@iJungleboy issue created as discussed

@jeremy-farrance
Copy link
Contributor

Strongly agree: View Edit | Add ...

Also, LOVE the icons, but I think you can skip the fancy tooltip and just set the element title for a more basic and standard hover info.

@valadas
Copy link
Contributor

valadas commented Oct 25, 2024

If work is done in that area, we could do it on dnn-elements permission grid (which is used in the resource manager) and then swap the react one to it.

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

3 participants