Posted by catch on June 4, 2009 at 10:37am
6 followers
| Project: | Drupal core |
| Version: | 8.x-dev |
| Component: | user.module |
| Category: | task |
| Priority: | normal |
| Assigned: | rocket_nova |
| Status: | needs review |
| Issue tags: | string freeze |
Issue Summary
#480660: Add an 'administrator' role to core just went in, the admin/user/permissions help text could do with updating in the meanwhile before we do #468768: Remove hardcoded anonymous and authenticated user roles.
This also clarifies the inheritance of permissions from authenticated user which doesn't appear to have ever been there. Was working on this with Leisa in #drupal-usability but Dries beat us to it.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| admin.patch | 2.47 KB | Idle | Unable to apply patch admin_13.patch | View details | Re-test |
Comments
#1
Yes, better tone of voice and more useful info
#2
Committed. Thanks!
#3
Hm, I like the text, but I think it might be confusing in the case where their administrator role is not actually named "Administrator". It doesn't really do anything to help people identify which role on the page is currently the administrator role...
It almost feels like an icon (+tooltip) would be a better way to communicate this, rather than help text? An icon could be used other places the role appears too (for example, the user edit page, where it might be useful to call out this role also). The only problem is that I have no idea what a universal icon for "administrator" would look like :)
Also, to nitpick: There seems to be some inconsistency in the capitalization, and probably change "log into" => "log in to" and "meaning this will" => either "meaning it will" or "meaning this role will"?
#4
String freeze, lets discuss this in D8
#5
If #576304: Three roles now, not two -> explain the Administrator role on the Roles page! is committed, then I think the confusion @David_Rothstein brings up in #3 is mitigated. In that case, I suggest that we remove all mention of the Administrator role and allow the help text on admin/people/permissions/roles to explain the Administrator role. Patch attached to that effect.