Problem/Motivation

Quoting @hooromoo from #3269657-11: [drupalMedia] The CKEditor 4 → 5 upgrade path for the media_embed filter should not forcefully allow the `data-view-mode` attribute on `<drupal-media>`

2. In Drupal\Tests\ckeditor5\FunctionalJavascript\AdminUiTest::testPluginSettingsFormSection I am not sure why the media plugin is enabled when the media_embed filter is disabled.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3276627

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

Assigned: Unassigned » wim leers

Reason this is being found now: #3269657: [drupalMedia] The CKEditor 4 → 5 upgrade path for the media_embed filter should not forcefully allow the `data-view-mode` attribute on `<drupal-media>` is making media_media the very first filter-dependent configurable CKE5 plugin.

Good find, @hooroomoo!

Working on test.

wim leers’s picture

Assigned: wim leers » Unassigned
Status: Active » Needs review
wim leers’s picture

Issue tags: -Needs tests
hooroomoo’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me, waiting for the tests pass now

  • catch committed b26aa84 on 10.0.x
    Issue #3276627 by Wim Leers, hooroomoo: CKEditor5::...

  • catch committed 54fc1ba on 9.4.x
    Issue #3276627 by Wim Leers, hooroomoo: CKEditor5::...
catch’s picture

Version: 9.4.x-dev » 9.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.0.x, cherry-picked to 9.4.x and 9.3.x, thanks!

  • catch committed b96392e on 9.3.x
    Issue #3276627 by Wim Leers, hooroomoo: CKEditor5::...

Status: Fixed » Closed (fixed)

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