Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.#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.
| Comment | File | Size | Author |
|---|---|---|---|
| #5 | permissions-helptext-481802-5.patch | 2.41 KB | drupal_was_my_past |
| admin.patch | 2.47 KB | catch | |
Comments
Comment #1
yoroy CreditAttribution: yoroy commentedYes, better tone of voice and more useful info
Comment #2
Dries CreditAttribution: Dries commentedCommitted. Thanks!
Comment #3
David_Rothstein CreditAttribution: David_Rothstein commentedHm, 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"?
Comment #4
aspilicious CreditAttribution: aspilicious commentedString freeze, lets discuss this in D8
Comment #5
drupal_was_my_past CreditAttribution: drupal_was_my_past commentedIf #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.
Comment #6
mgifford#5: permissions-helptext-481802-5.patch queued for re-testing.
Comment #24
rocketeerbkw CreditAttribution: rocketeerbkw as a volunteer commentedThe original patch was already committed and the issues mentioned in #481802-3: Update user permissions help text have either been addressed in the last 13 years or there are follow up issues for them. The patch in #481802-5: Update user permissions help text is also outdated, so I'm closing this.
Follow ups:
Comment #25
rkollerAdded the Bug Smash Initiative tag
Comment #26
quietone CreditAttribution: quietone at PreviousNext commented@rkoller, this issue has never been a Bug Report, which is the Bug Smash tag added?
Comment #27
rkolleruh sorry. it was a misunderstanding on my end. after @rocketeerbkw mentioned the closed issue in the #bugsmash channel and also got emote reaction in agreement i thought i would fall under the bug smash category. but i forgot it is bugs only as the name implies and also brandon state by "in the spirit of". my bad sorry. i've removed the tag again. thanks for the headsup and clarification!