Openrct2: The default permission window is too easy to misclick

[Nubbie]

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]

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]

Here are my suggestions;

Suggestion 1:

Suggestion 2:

[marijnvdwerf]

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

[Nubbie]

@marijnvdwerf
?

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]

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]

@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]

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

[marijnvdwerf]

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

Was thinking of something like this:

'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]

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]

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

[janisozaur]

@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]

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]

@Nubbie there's little accountability right now, if that was to change by means of https://github.com/OpenRCT2/OpenRCT2/pull/4273 they wouldn't care that much about this window.

[marijnvdwerf]

@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]

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

[janisozaur]

It looks nice, but I think it could use two minor improvements:

• The "make default" should be a button, rather than checkbox
• The default group should get some highlight/tick/marker next to it in the groups' list.

[marijnvdwerf]

By using a checkbox for 'default group', you can get rid of the 'block unknown players' option.