The default permission window is too easy to misclick #5033

Open
Nubbie opened this Issue Jan 10, 2017 · 15 comments

Projects

None yet

4 participants

@Nubbie
Contributor
Nubbie commented Jan 10, 2017 edited

Commit/Build: 06d1d21

Suggested on NEDesign by Liampie

  • In the permissions window, the 'default user group' is not hidden away well enough. It has happened on multiple occasions that someone accidentally set the default user group to the most powerful group, which is naturally very dangerous. It's too close to that other drop-down menu with all the user groups, easy to confuse. I think you should move it elsewhere!

Indeed it's a bit too easy to missclick and change, a prompt or re-adjustment of it could be useful

Edit; Have spoken a bit with him and will try to make a PR

@janisozaur
Member

I have been affected by this myself. The simplest reordering of the window's widgets to have default group at the bottom should help, but @marijnvdwerf can surely offer a better solution.

@Nubbie
Contributor
Nubbie commented Jan 10, 2017

Here are my suggestions;

Suggestion 1:
idea 2

Suggestion 2:
sss

@marijnvdwerf
Member

Why not move the "default group" to the settings tab, and use some "{STRINGID} (default)" in the group dropdown?

@Nubbie
Contributor
Nubbie commented Jan 10, 2017 edited

@marijnvdwerf
?
ischy

Something like this was thought about - but it might confuse people from where to change it as it's in a whole different spot
Note the text can be removed and just use "{STRINGID} (default)" indeed, might be easier, but my worried point still stance :v

@marijnvdwerf
Member

I find the entire dialog pretty confusing at the moment. I'd say a two-column design with a richter detail pane would make a lot more sense, but thats a bit much with the current window system.

Anhow, "Block unknown players" actually is the same as "Default group", so I'd say that it makes perfect sense to have those options together.

@janisozaur
Member

@marijnvdwerf I was also thinking dual-pane design. This would allow quicker access to group's settings, where in left pane there are groups, in right pane its permissions.

There could be even third pane with all the players in currently selected group.

@Nubbie
Contributor
Nubbie commented Jan 10, 2017

Was designing something like Suggestion 1 but presume it's not that interesting as it might need to be discussed a bit ;o

@marijnvdwerf
Member

A third pane wouldn't make sense to me, unless you're going for drill-down navigation.

Was thinking of something like this:
image

'Delete' would be a red-coloured button at the bottom of each detail pane. Members would be draggable to groups, as well as clickable to open a user window. Top of the pane would contain the name + edit control, followed by a 'new users join this group by default' checkbox. If another checkbox is selected, the default group changes. If the active group has it's checkbox unselected a warning shows up that 'this will block new users from accessing the server'.

Only thing missing is a 'new group' button.

@Nubbie
Contributor
Nubbie commented Jan 10, 2017 edited

At the same time - People might only want to look at one window at the time (many admins check just users etc this way) with as little to obscure as possible, if the other two options are hide-able, it might work

@marijnvdwerf
Member

Not sure what user checking has to do with group management. What use case are you talking of, exactly?

@janisozaur
Member

@marijnvdwerf users don't have any avatars, so the circles you've drawn are not something we can do. We only have usernames to make use of.

@Nubbie
Contributor
Nubbie commented Jan 10, 2017 edited

Example picture
The thing is that some admins/hosts want the player panel as small as possible so you can watch more players and be able to kick the person acting bad faster & being able to see what commands everyone is doing
If the two other panels are retractable, it wouldn't obscure more which would make that proposal interesting in a way
If the user list isn't re-worked, this wouldn't be a problem :)

@janisozaur
Member

@Nubbie there's little accountability right now, if that was to change by means of #4273 they wouldn't care that much about this window.

@marijnvdwerf
Member

@janisozaur You could also do a textual list. But I just found that visually boring. Was more of a quick sketch than anything 100% serious.

@Nubbie This would only be a replacement for the 'group management' part of the window (and the 'block users' setting). The user list wouldn't be touched.

@marijnvdwerf
Member

Update: quick sample thingy of what I mean: https://jsbin.com/coholasoye/edit?html,css,output

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment