Recreating this issue that currently exists in core: https://www.drupal.org/project/drupal/issues/3092593

I'm guessing this needs to be fixed here if Drupal core has switched to depending on contrib CKEditor?

Original summary is below: ⬇️

Problem/Motivation

(for quick setup, Install Umami 8.8.x)

Steps to reproduce -

1. View an article, and click on quickedit on top-right corner
2. Place the cursor anywhere in the body, and click on add-media icon
3. Select the image from media_library
4. Click on "Insert Selected"

Result -
Image is not added, and there's an error in the console

plugin.js?t=0:148 Uncaught TypeError: Cannot read property 'getAttribute' of null
    at d.upcast (plugin.js?t=0:148)
    at Array.<anonymous> (<anonymous>:29:162)
    at iterator (<anonymous>:874:440)
    at window.CKEDITOR.window.CKEDITOR.dom.CKEDITOR.htmlParser.element.forEach (<anonymous>:294:397)
    at window.CKEDITOR.window.CKEDITOR.dom.CKEDITOR.htmlParser.fragment.forEach (<anonymous>:294:498)
    at e.<anonymous> (<anonymous>:887:94)
    at e.g (<anonymous>:10:70)
    at e.<anonymous> (<anonymous>:11:433)
    at e.window.CKEDITOR.window.CKEDITOR.dom.CKEDITOR.editor.CKEDITOR.editor.fire (<anonymous>:13:97)
    at CKEDITOR.htmlDataProcessor.toHtml (<anonymous>:315:222)

https://www.youtube.com/watch?v=nOCuBPgNink

CommentFileSizeAuthor
#6 3341001_6.patch1.48 KBzhawkins
#5 3341001_3.patch1.48 KBzhawkins

Issue fork ckeditor-3341001

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

ZHawkins created an issue. See original summary.

zhawkins’s picture

Issue summary: View changes

zhawkins’s picture

StatusFileSize
new1.48 KB
zhawkins’s picture

StatusFileSize
new1.48 KB

Fixing the patch file number.

firewaller’s picture

Status: Active » Reviewed & tested by the community

Patch #6 works for us.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 6: 3341001_6.patch, failed testing. View results