Problem/Motivation

When an admin selects "Use the administration theme when editing or creating content" on the Appearance page (see uploaded screenshot), it is not clear that only certain roles (the admin role) will be able to view the administration theme when editing or creating content. It is obvious to most of us because we've been doing this for a while, but it isn't clear to new users/admins.

Proposed resolution

Add a description beneath "Use the administration theme when editing or creating content" to tell users/admins that they can control which users can view the administration theme on the Permissions page.

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

jordanpagewhite created an issue. See original summary.

jordanpagewhite’s picture

I added a description beneath "Use the administration theme when editing or creating content" on the admin Appearance page. See patch and screenshot.

jordanpagewhite’s picture

Status: Active » Needs review

Needs review

pguillard’s picture

Status: Needs review » Reviewed & tested by the community

I wonder if it would be even better to add a link on "Permissions Page".
But it is RTBC for me.

jordanpagewhite’s picture

@pguillard That is an interesting suggestion. The Permissions page already has this description, "This is only used when the site is configured to use a separate administration theme on the Appearance page.", beneath this option, "View the administration theme". I feel that is a sufficient 'alert' to admins that aren't already aware of this, but maybe I am biased because I've been doing this for a while.

My hopes, with this issue, was to avoid any potential confusion for an admin who toggled the "Use the administration theme when editing or creating content" since there is no description/alert/warning to an inexperienced admin on that page. Thank you for taking the time to review the patch. I really appreciate it.

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

I can confirm the issue. I think the help text could link to the permissions page and I think the permission needed, "View the administration theme", should be quoted.

jordanpagewhite’s picture

Status: Needs work » Needs review
FileSize
814 bytes
922 bytes

Updated with suggestions. Thanks everyone for reviewing. I really appreciate it.

webbykat’s picture

(Originally commented without the patch, corrected below.)

webbykat’s picture

Awesome idea - this has always nagged me too. This patch did work for me. I've suggested a minor wording update to connect the two things that must be done (checking that box, *and* granting permission) so that the option below now says:

Control which roles can "View the administration theme," if the option above is checked, on the Permissions page.

jordanpagewhite, what do you think?

jordanpagewhite’s picture

@webbykat It seems to me like that might be redundant because of the phrase right above: "Use the administration theme when editing or creating content".

pguillard’s picture

I would suggest to replace @ with : for placeolder keys. Somebidy can confirm ?
Otherwise, this is RTBC for me.

jordanpagewhite’s picture

pguillard, You are correct. I've attached an updated patch and interdiff. See API docs for further explanation https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Component%21Rend...

pguillard’s picture

Status: Needs review » Reviewed & tested by the community

  • catch committed b04e99a on 8.1.x
    Issue #2651228 by jordanpagewhite, webbykat: Add description beneath '...
catch’s picture

Version: 8.0.x-dev » 8.1.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.1.x, thanks!

Status: Fixed » Closed (fixed)

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