We can definitely incorporate changes, feel free to continue posting here. I just wanted to signal to the watchers that this feature is functionally complete and ready for review and testing. It will also be included in the upcoming release.

Hm. As you have marked this as resolved, there will be no changes? At least the menu name and the title on the page should be the same, to be consistent.

Happy to change "Access Controls" to "Permissions" in the English version. Note also that there will be some additional work needed at some point to update the other (non-English) localizations updated. Assistance from the community is welcome for that effort.

This is now feature-complete and will be available in the nightly build starting tomorrow.

Do we need to change the name to Access Controls (the name is wonky)? What's wrong with User Permissions? User is a member of a group, so it is still about user permissions even if you can add a group of users to some role. Also it says "Room Affilations" on the page itself, so there is a disconnect between menu name and the title in there. Affilations is probably a more suitable word in this case, but many non-english speakers would not know it. Permissions is a more common name.

Other than that, it works. Though i don't really like that window with a scrollbar (which will get clunky with hundreds of groups), but i don't know of a better solution. Paged list of groups would also be clunky and having to type group names also inconvenient.

PR submitted and ready for review: https://github.com/igniterealtime/Openfire/pull/100