This module appears to be working as billed.
One minor suggestion, it may be more understandable if the project page documentation uses an example that includes a checkbox for "authenticated" user as that is one that many people will be able to relate to and easily understand the applicability of this module.
Thanks,
Izzy
Comments
Comment #1
izmeez commentedmarc,
I can't help myself thinking more about this new module. Please just accept this as brainstorming.
I wonder if it might be helpful for the UI to say,
"Also make visible to other users not in these groups that are:
[ ] authenticated users
[ ] students
[ ] teachers
etc.
It is possible that some users might otherwise misunderstand that the inclusion of these roles are sub-groups of the group but rather in addition to the group.
Thanks,
Izzy
Comment #2
marcp commented@izmeez - Yes, I agree that the wording on there needs work.
There may be times when the Group Audience isn't shown but the Role Audience is shown. The wording should make sense regardless.
How about:
Title: This post should be visible to users in the following roles:
[ checkboxes here ]
[ checkboxes here ]
[ checkboxes here ]
Description: Users in the selected roles will be able to view this post regardless of their group memberships.
?
Comment #3
izmeez commented@marc good points, sparks two thoughts.
For Title, is it better to use "should be visible" or "make visible"?
I must admit I lean towards the latter and it's shorter.
For Description, is there some way to combine this with the title or place it as a subtitle but keep it short. I find it awkward when useful hints are at the bottom of the list if the list has more than a few items.
How about:
Make visible to all users in the following roles regardless of group membership.
[ ] role1
[ ] role2
[ ] role3
Also, please consider adding the check box for "authenticated users" in the example screen display on the project page. I think it will help readers understand the functionality of this module.
Thanks, this is a great module.
Comment #4
marcp commentedI've been playing with this a bit and think that a nice hybrid would be:
Title: Make visible to all users in the following roles:
[ ] role1
[ ] role2
[ ] role3
Description: Users in the selected roles can view this post regardless of their group memberships.
As far as putting the description underneath the title, that's really the responsibility of the theme -- I agree that seeing the description under a whole mess of checkboxes is bad news, but that's a separate issue.
Comment #5
izmeez commentedlooks good.
Comment #6
marcp commentedIt's not the greatest, but the image and project documentation have been updated. If anyone's got a better description of the "authenticated user" use-case I'd love to hear it and adjust things so that it makes sense to a larger audience.