Consider using the administration theme when editing or creating content by default

Tagging for framework manager for their thoughts.

Personally I would not like to see this happen as I think this goes against the work done for claro.

Personally I would not like to see this happen as I think this goes against the work done for claro.

How so? Changing this would make it so Claro is used in even more situations, right?

Having a terrible case of the Mondays and no coffee. You are correct, some reason I read it as using the front end theme by default.

That's my mistake. Will mark it and see what the committer says.

I think Drupal may have been a system where using the frontend theme was the sensible default, but its used for more ambitious projects now and using the admin theme by default fits with that better. (Content types are more complex and usually other admin tasks are involved in creating the content, such as category management or media management).

I don't think this actually needs a framework manager review so removing the tag.

Something we still need to do is to go through tests where $this->config('node.settings')->set('use_admin_theme', '1') is being called to make sure that we are not changing any tests to not test what they are supposed to test.

Yeah it'd be good to know if we have explicit test coverage around this setting. Plus we'll need a CR to announce the change to people.

+1 for the change.

Searching for

->config('node.settings')->set('use_admin_theme',

There are 4 instances and 3 I believe can be removed.

ConfigurableLanguageManagerTest
AddedStylesheetsTest
ShortcutLinksTest

As far as a dedicated test where would that go?