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

fix(ck5): respect options passed to data controller on set #886

Closed
wants to merge 2 commits into from

Conversation

jonscheiding
Copy link

Description

Make monkey-patched data.set and data.get respect options (and any other arguments that may be added some day)

Closes #885

@xjiang-at-wiris xjiang-at-wiris self-requested a review March 1, 2024 11:18
@jguillen-at-wiris jguillen-at-wiris removed the request for review from xjiang-at-wiris March 4, 2024 19:10
@jonscheiding
Copy link
Author

jonscheiding commented Mar 5, 2024

@jguillen-at-wiris It actually occurred to me that I should switch this to use the event mechanism that CKE5 offers for stuff like this, that keeps it from interfering with other plugins that are also trying to use that mechanism. Just pushed a second commit.

@carla-at-wiris
Copy link
Contributor

This has been included on our latest release. Check the following Pull Request for more information #958

Thank you for your contribution!

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

Successfully merging this pull request may close these issues.

[CKEditor 5] options passed to setData are ignored
3 participants