revolution icon indicating copy to clipboard operation
revolution copied to clipboard

Simplify the “User Groups” section: display groups as a grid, not a tree

Open Ruslan-Aleev opened this issue 4 years ago • 15 comments

Feature request

Summary

In the current version, it’s almost impossible to understand where to edit a specific “User Group”, because the editing link is hidden in the context menu. And when you click on a group, it’s not the group’s settings appearing (what you expect), but adding a user to the group.

users_groups_1

users_groups_2

I suggest making the display of "User Group" in the form of a grid instead of a tree. And the user can be added to the group inside the group editing (this already works, see gif above). This will be much clearer, especially with the new link on name and gear context menu.

It will look something like this:

users_groups_3

It will be possible to display useful information in the "User Group" grid, such as users in a group, access to sections (contexts, groups, etc.)

Why is it needed?

Simplification in UI / UX

Related issue(s)/PR(s)

N/A

Ruslan-Aleev avatar Jul 04 '20 15:07 Ruslan-Aleev

In addition

  1. Rename the "User groups & users" tab to "User groups".
  2. Simplify the copy logic of Access Policies .

sergant210 avatar Jul 04 '20 16:07 sergant210

I agree that it's confusing that way it's set up now.

SnowCreative avatar Jul 15 '20 23:07 SnowCreative

A "Update User group" or "Update group settings" button above the already loaded grid would also make it much clearer.

sebastian-marinescu avatar Jul 16 '20 16:07 sebastian-marinescu

A "Update User group" or "Update group settings" button above the already loaded grid would also make it much clearer.

I do not quite understand. Now all actions are located in the gear menu on the right, so an additional button is not needed.

Ruslan-Aleev avatar Jul 16 '20 17:07 Ruslan-Aleev

Your suggestion and UI proposal look very good, assuming someone would provide a PR for that. I'm just saying that it might also be possible to add another button, like this:

image

sebastian-marinescu avatar Jul 16 '20 19:07 sebastian-marinescu

Now it's clearer :) Yes, we can do this, and it's easier. But personally, I am in favor of rewriting the section completely. p.s. Probably for branch 2.8 the button solution would be appropriate.

Ruslan-Aleev avatar Jul 16 '20 20:07 Ruslan-Aleev

I'm ok with having a button in 3.0 as well. A rewrite would be nice but it's something that can go into 3.1 as well.

JoshuaLuckers avatar Jul 17 '20 07:07 JoshuaLuckers

I was also always confused by the name User Groups, isn't it more logical to call it Access Groups?

Ruslan-Aleev avatar Jul 28 '20 20:07 Ruslan-Aleev

Not necessarily; there are User Groups and Resource Groups. Access Groups could easily get confused for either of those.

sottwell avatar Jul 28 '20 21:07 sottwell

I was also always confused by the name User Groups, isn't it more logical to call it Access Groups?

Sometimes we group Users for other reasons than access. So User Groups is more logical for me.

Bruno17 avatar Jul 29 '20 05:07 Bruno17

The button has been added in 2.8 but I'll leave the issue open if people want to discuss further how users & user groups is currently set up with the tree.

Mark-H avatar Oct 04 '20 20:10 Mark-H

Improved in #15270

@modxbot close

Ibochkarev avatar Oct 22 '20 04:10 Ibochkarev

@modxbot open

Ibochkarev avatar Oct 22 '20 07:10 Ibochkarev

In the future, it makes sense to remove other trees as well, if possible. Trees (other than resources and menus) are controversial UX choices, see https://github.com/modxcms/revolution/issues/10518, https://github.com/modxcms/revolution/pull/15478

Ruslan-Aleev avatar Mar 05 '21 13:03 Ruslan-Aleev

This idea would be a no-brainer to me ... except ... how would we handle the nesting of user groups (which is allowable and easy to show/manage in the tree view)? I haven't been called on by any of my modx clients to set up user groups where subcategorization would be beneficial; as such, the proposed simplification would not be a problem for me. For others it could be problematic.

Anyway, something to think about before someone dives into a PR at some point...

smg6511 avatar Jul 08 '22 21:07 smg6511