Problem/Motivation

(Sorry for the indescriptive issue title, it's late...)

When two CKEditor instances are loaded on the same page with the same format, if the first has a "readonly" attribute, but the second does not, they both get initialized in readOnly-mode.

Proposed resolution

?

Remaining tasks

User interface changes

API changes

CommentFileSizeAuthor
#2 ckeditor_readonly-2291439-2.patch903 byteswim leers
#1 ckeditor-readonly-again.patch651 byteststoeckler

Comments

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
commented
StatusFileSize
new ckeditor-readonly-again.patch651 bytes

Here's a patch that works, but it feels pretty hacky.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Title: CKEditor's "readonly" is too sticky » CKEditor's "readonly" needs to be per-instance but is applied per-format, across instances
Status: Active » Reviewed & tested by the community
Issue tags: +CKEditor in core, +JavaScript
Related issues: +#2275491: CKEditor does not support the "readonly" attribute, +#2276187: Remove the CKEditor readonly attribute support work-around
StatusFileSize
new ckeditor_readonly-2291439-2.patch903 bytes

Great catch, thanks! Only rerolled to update the code comment accordingly.

No matter what, this is going to be hacky until it's fixed upstream, and then we can remove that entire code block anyway. We're tracking that at #2276187: Remove the CKEditor readonly attribute support work-around.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Committed a22e231 and pushed to 8.x. Thanks!

  • alexpott committed a22e231 on 8.x 
    Issue #2291439 by Wim Leers, tstoeckler: Fixed CKEditor's "readonly"...

Status: Fixed » Closed (fixed)

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