Problem/Motivation

Splitting this off from #1044090-6: Enable toolbar for authenticated users also, so that non-admin users can use shortcuts as well point 2.

The toolbar doesn't have to be just for "administration". Especially once #2135445: Toolbar displays Manage tab even if the user is not permitted to see it is fixed, toolbar is very useful for content editors or even regular authenticated users, as a slick/easy way to access shortcuts, to have a nicer user menu experience, and potentially other links/trays added via contrib.

However, while the machine name for the permission to use the toolbar is access toolbar, the human-readable label in the admin UI is unfortunately called "Use the administration toolbar". This implies the toolbar is for "administration", and that granting this permission to non-admins is a bad idea. This is not true (or at least, won't be true once #2135445 is committed).

Proposed resolution

Rename the label to just "Use the toolbar".

Machine name of the permission (access toolbar) remains unchanged. Therefore, no impact on existing installs, no upgrade path required, etc.

The only impact is a minor translation break for this permission label. Should be an easy fix for translators, since they only have to remove something from the existing translations.

Remaining tasks

  1. Upload the trivial patch.
  2. Get sign-off from the appropriate folks (usability team? anyone else?)
  3. Decide if we need a change record, and if so, write it. Draft text in the Release notes section below
  4. Commit

User interface changes

A small change to the human-readable label for toolbar's one and only permission, to remove the unnecessary word "administration".

API changes

None.

Data model changes

None. The machine name for the permission is unchanged. No impact on existing or future installs.

Release notes snippet

The permission to use the toolbar has been renamed from "Use the administration toolbar" to "Use the toolbar". The toolbar does not require a user have permissions to perform any "administration" tasks, so the permission label has been simplified to make it clear to site builders that the toolbar could be enabled for various non-administrator roles. The machine name for this permission (access toolbar) remains as-is, so no changes are required to existing site configurations.

CommentFileSizeAuthor
#2 3025839-2.patch341 bytesdww
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dww created an issue. See original summary.

dww’s picture

Status: Active » Needs review
FileSize
341 bytes

I grepped the whole 8.7.x tree. No mentions of this permission label anywhere except where it's defined.

dww’s picture

Issue summary: View changes

Update Remaining tasks.
Minor clarification in the "Release notes snippet".

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs usability review

The module is also simply named "Toolbar" and not "Administration Toolbar" so this would indeed be more consistent. I don't think this warrants a usability meeting spot.

dww’s picture

Issue summary: View changes

Thanks! Updated remaining tasks accordingly.

Cheers,
-Derek

Wim Leers’s picture

Issue tags: +Usability

RTBC +1

Wim Leers’s picture

Issue tags: +String change in 8.7.0
Wim Leers’s picture

larowlan’s picture

Status: Reviewed & tested by the community » Fixed

Created a change record and published it.

Committed 5ba43ae and pushed to 8.7.x. Thanks!

  • larowlan committed 5ba43ae on 8.7.x
    Issue #3025839 by dww: Change toolbar's permission label to 'Use the...
dww’s picture

Thanks! Glad the draft text was useful.

Cheers, -Derek

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.