Editor and Filter configuration from drupal_cms_starter is not applied

Created on 16 November 2024, about 1 month ago

Problem/Motivation

The config for basic_html editor profile contained in drupal_cms_starter does not appear to be applied at install. Possibly some of the other configuration there is not installed either.

This blocks using media effectively in CKEditor fields.

Steps to reproduce

Install an instance of Drupal CMS.

Navigate to /admin/config/content/formats/manage/basic_html

Inpsect filter.format.basic_html.yml and
editor.editor.basic_html.yml in the drupal_cms_starter recipe.

Compare the UI with the settings in the yaml file and also try to use CKEditor to add some content. Notice, for example, that media is not active on the toolbar, instead blockquote and image upload (and others) are included and should not be.

Optionally, try to export some configuration to update these files. No matter the contents of the yml, they are not included at install.

Reporter attempts to resolve

I scanned other recipes and in case something is overwriting it after this config is applied, I scanned recipes for references to editor files and tried to debug by removing editor dependencies from recipes.

I concluded that these files are likely ignored.

I notice that there is no reference to this particular config in the recipe.yml of drupal_cms_starter - I tried adding some yml but didn't successfully install these settings. I tried setting strict to true.

Proposed resolution

We need some way to set and control the editor config state at install. `

First, please verify the issue.

Possibly, the configuration should be moved to another recipe, or it needs to be written with config actions (how does the format look?), or some other treatment.

Once we have found the right way to handle the config, we should check other configuration that may not have installed and if we get it to install, make sure it works as intended (if it hasn't imported for a while, it may not longer be needed or wanted).

πŸ› Bug report
Status

Active

Component

Base Recipe

Created by

πŸ‡¬πŸ‡§United Kingdom tonypaulbarker Leeds

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @tonypaulbarker
  • πŸ‡ΊπŸ‡ΈUnited States phenaproxima Massachusetts

    What I suspect is happening here is that another one of the recipes in the starter's stack is applying core's basic_html recipe (which is necessary in many cases for default content). Since the starter doesn't use config actions to set up its extensions to basic_html, but instead ships it wholesale in the config directory, it gets ignored so as not to conflict with the already-existing config.

    What needs to happen here is one of two things:

    The starter (or some other recipe in its stack) needs to use config actions to alter the basic_html format from core.
    The starter needs to ship a different format that has the needed changes - this is the approach being taken in πŸ“Œ Create a new text format/editor to replace core's Basic HTML Active .

  • πŸ‡¦πŸ‡ΊAustralia pameeela

    Is there a problem with πŸ“Œ Create a new text format/editor to replace core's Basic HTML Active ? Would love to get that merged ASAP since it fixes this and other issues. Should really get it in before beta.

  • πŸ‡¬πŸ‡§United Kingdom tonypaulbarker Leeds

    @pameeela @phenaproxima thank you. That seems like the right way forward! Issue 3486284 is a different framing of the same problem and we should put our efforts to getting that done - we can extend that with config actions too if we want - so I’ll close this as a duplicate.

Production build 0.71.5 2024