Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CKEditor 5 does not have a read only setter #4

Open
IsAvaible opened this issue May 21, 2022 · 2 comments
Open

CKEditor 5 does not have a read only setter #4

IsAvaible opened this issue May 21, 2022 · 2 comments

Comments

@IsAvaible
Copy link

Problem
Line 42 of Ckeditor.svelte tries to set the read-only status with this piece of code: editor.isReadOnly = disabled;.
This is not supported by CKEditor 5, because read-only has no setter. Read more here.

Screenshot of the error message
image

Suggested Solution
Since I don't see the purpose of disabling the editor anyway, I would suggest removing the line altogether.

@jecovier
Copy link
Contributor

jecovier commented Jun 4, 2022

@IsAvaible that's true, also if you need to disable de editor you can always do that on the onReady function. Anyways, I fix it in my project using the new CKEditor method enableReadOnlyMode( ), @techlab23 I created a PR with that fix.

Repository owner deleted a comment from tobarbaro Feb 10, 2024
Repository owner deleted a comment from tobarbaro Feb 10, 2024
Repository owner deleted a comment from tobarbaro Feb 10, 2024
@th3c0r
Copy link

th3c0r commented Aug 22, 2024

Hi, I tried to use this package in my project and encountered the same error, even though I see that it was corrected in commit 2733412

I installed the package directly from the GitHub URL, and it worked fine, unlike the version installed via npm. It seems the npm package needs to be updated to include the latest changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants
@jecovier @th3c0r @IsAvaible and others